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

Updating Visual Diff Goldens for PR 3881 #3882

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 8, 2023

This pull request updates the visual-diff golden images for the changes in pull request #3881.

Please review the following failed reports closely before merging to ensure the visual differences are expected.

Failed Reports:

@github-actions github-actions bot requested a review from svanherk August 8, 2023 18:20
@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
@github-actions github-actions bot force-pushed the ghworkflow/visual-diff-pr-3881 branch from 1759398 to 6e0dab7 Compare August 11, 2023 18:59
@github-actions github-actions bot force-pushed the ghworkflow/visual-diff-pr-3881 branch from 6e0dab7 to 0857d7b Compare August 11, 2023 19:14
@svanherk svanherk marked this pull request as ready for review August 11, 2023 19:22
@svanherk
Copy link
Contributor

Looks as I expected - slight changes from changing the padding, and some new diffs that don't look great (but that's the point!)

@svanherk svanherk merged commit 81212d6 into DE54475_Add_test_demos_show_sticky_list_controls_scroll_issue Aug 11, 2023
@svanherk svanherk deleted the ghworkflow/visual-diff-pr-3881 branch August 11, 2023 19:23
svanherk added a commit that referenced this pull request Aug 21, 2023
* Add list visual-diffs and dialog demos to show the sticky list controls + scroll issue

* Updating Visual Diff Goldens (#3882)

Co-authored-by: github-actions <[email protected]>

* Fix sticky d2l-list-controls width

* Updating Visual Diff Goldens (#3900)

Co-authored-by: github-actions <[email protected]>

* Ignore length-zero-no-unit style linting rule for css variable that will be used in calc

* Avoid having to set the css variable in Javascript

* Revert Avoid having to set the css variable in Javascript

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions <[email protected]>
svanherk added a commit that referenced this pull request Aug 21, 2023
* Add list visual-diffs and dialog demos to show the sticky list controls + scroll issue

* Updating Visual Diff Goldens (#3882)

Co-authored-by: github-actions <[email protected]>

* Fix sticky d2l-list-controls width

* Updating Visual Diff Goldens (#3900)

Co-authored-by: github-actions <[email protected]>

* Ignore length-zero-no-unit style linting rule for css variable that will be used in calc

* Avoid having to set the css variable in Javascript

* Revert Avoid having to set the css variable in Javascript

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions <[email protected]>
(cherry picked from commit 6b6501e)
svanherk added a commit that referenced this pull request Aug 21, 2023
)

(This is a backport of #3899)

* Add list visual-diffs and dialog demos to show the sticky list controls + scroll issue

* Updating Visual Diff Goldens (#3882)

Co-authored-by: github-actions <[email protected]>

* Fix sticky d2l-list-controls width

* Updating Visual Diff Goldens (#3900)

Co-authored-by: github-actions <[email protected]>

* Ignore length-zero-no-unit style linting rule for css variable that will be used in calc

* Avoid having to set the css variable in Javascript

* Revert Avoid having to set the css variable in Javascript

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions <[email protected]>
(cherry picked from commit 6b6501e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant