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 vdiff goldens for PR 3986 #3987

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 7, 2023

This pull request updates the vdiff goldens for the changes in pull request #3986.

Please review the resulting report closely before merging to ensure the visual differences are expected.

@svanherk
Copy link
Contributor

svanherk commented Sep 8, 2023

Expected changes:

  • click left arrow and click right arrow - there's now click styles on the arrow because we're using proper click interactions now and the click style is applied from clicking it
  • click disabled - a fix was added to the PR to remove focus styles on disabled dates, but there's still a change because we click on the disabled value and move focus
  • initial focus date is 1st of month and pagedown - the mouse is still hovering over the arrow from when we clicked it. We could move the mouse if we care.

@dlockhart dlockhart force-pushed the US152268_Migrate_calendar_vdiff_tests branch from 1df2546 to fb38fc3 Compare September 8, 2023 17:51
@dlockhart
Copy link
Member

These look good now -- the Arabic golden was incorrect as it was only setting the page language and not the direction previously whereas now the table cells are correctly reversed.

@svanherk svanherk marked this pull request as ready for review September 11, 2023 14:07
@svanherk svanherk merged commit da02bb7 into US152268_Migrate_calendar_vdiff_tests Sep 11, 2023
@svanherk svanherk deleted the ghworkflow/vdiff-pr-3986 branch September 11, 2023 14:07
svanherk added a commit that referenced this pull request Sep 12, 2023
* US152268 - Migrate calendar vdiff tests
* Move existing goldens to new location
* Fix the focus effects for disabled dates (other than selected) (#3989)
* fix: remove caching of calendar date/time descriptor
* Updating vdiff goldens (#3987)

Co-authored-by: github-actions <github-actions@github.com>

---------

Co-authored-by: Dave Lockhart <dlockhart@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions <github-actions@github.com>
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.

2 participants