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

fix(linux): update outdated snapshots after release 2.4 #144

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

daniel-herrero
Copy link
Collaborator

@daniel-herrero daniel-herrero commented Dec 11, 2024

This PR updates snapshots for linux after the release 2.4, fixing mainly the new masks.

@daniel-herrero daniel-herrero force-pushed the dherrero/bugfix/linux-snapshots-2.4 branch from 4462241 to 30be4d2 Compare December 12, 2024 08:02
@daniel-herrero daniel-herrero changed the title [WIP] fix(linux): update outdated snapshots after release 2.4 fix(linux): update outdated snapshots after release 2.4 Dec 12, 2024
@daniel-herrero daniel-herrero marked this pull request as ready for review December 12, 2024 09:38
Copy link
Collaborator

@tdelatorre tdelatorre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some changes in the snapshots of the view-mode test from dark mode to light mode (both Chrome and Firefox).
I checked the same snapshots on Win32 and all of them are in dark mode, so I think they are incorrect in light mode.
Sometimes after running the light mode tests, the test does not recover to dark mode and the snapshots are generated incorrectly.
It is recommended to review all the generated snapshots.

@daniel-herrero daniel-herrero force-pushed the dherrero/bugfix/linux-snapshots-2.4 branch from 30be4d2 to d5fcae2 Compare December 16, 2024 07:55
@daniel-herrero daniel-herrero force-pushed the dherrero/bugfix/linux-snapshots-2.4 branch from d5fcae2 to 08206a0 Compare December 16, 2024 08:05
@daniel-herrero
Copy link
Collaborator Author

All the light-dark-mode issues should be fixed by now. Well caught!

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.

2 participants