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 for regression from PR 43853 that didn't obey minimized and fullscreen windows #43894

Merged
merged 3 commits into from
Jan 10, 2024

Conversation

whimboo
Copy link
Contributor

@whimboo whimboo commented Jan 9, 2024

Fixes the regression from PR #43853.

CC @lutien

Copy link
Contributor

@lutien lutien left a comment

Choose a reason for hiding this comment

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

Thanks! Fixed the issue for me locally

@whimboo
Copy link
Contributor Author

whimboo commented Jan 9, 2024

There seems to be an issue with maximized as well. I would suggest to check for that as well, and actually make all the additional logic Chrome only given that for both Safari and Firefox the problem doesn't appear.

@whimboo whimboo merged commit d2b7ac0 into web-platform-tests:master Jan 10, 2024
18 of 19 checks passed
Nevalynn88

This comment was marked as spam.

@whimboo whimboo deleted the window_restore branch January 10, 2024 07:36
marcoscaceres pushed a commit that referenced this pull request Feb 23, 2024
…d and fullscreen windows (#43894)

Fix for regression from PR 43853 that didn't obey minimized, maximized and fullscreen windows.
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.

6 participants