Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue #3881

Conversation

svanherk
Copy link
Contributor

@svanherk svanherk commented Aug 8, 2023

Setting up the visual-diff list tests and dialog demo to show the issue, which will be fixed in a follow-up PR.

I was having a horrible time trying to get hover working properly for lists in dialog with the old visual-diff. I'm going to put up a PR moving the dialog tests to the new vdiff.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://pr-3881-brightspace-ui-core.d2l.dev/

Note
The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Visual diff tests failed - pull request #3882 has been opened with the updated goldens.

@svanherk svanherk changed the title DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue IGNORE FOR NOW DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue Aug 8, 2023
@svanherk svanherk changed the title IGNORE FOR NOW DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue IGNORE FOR NOW - DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue Aug 8, 2023
@svanherk svanherk force-pushed the DE54475_Add_test_demos_show_sticky_list_controls_scroll_issue branch from 3f33cef to 4d81420 Compare August 11, 2023 18:45
@svanherk svanherk changed the title IGNORE FOR NOW - DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue DE54475 - Add test and demos to show sticky d2l-list-controls scrolled + list hover/focus issue Aug 11, 2023
@svanherk svanherk marked this pull request as ready for review August 11, 2023 20:26
@svanherk svanherk requested a review from a team as a code owner August 11, 2023 20:26
@svanherk svanherk closed this Aug 17, 2023
@svanherk svanherk deleted the DE54475_Add_test_demos_show_sticky_list_controls_scroll_issue branch August 17, 2023 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant