Skip to content

Refactor testResizeWindow; increase window size for MSEdge#84

Merged
stof merged 2 commits intominkphp:masterfrom
uuf6429:improve-resize-window
Oct 17, 2023
Merged

Refactor testResizeWindow; increase window size for MSEdge#84
stof merged 2 commits intominkphp:masterfrom
uuf6429:improve-resize-window

Conversation

@uuf6429
Copy link
Copy Markdown
Member

@uuf6429 uuf6429 commented Sep 27, 2023

Main motivation for this: Edge most of the time fails this test because 500x300 is (apparently) too small.

  1. I did a small refactor of the JS side, so it's clearer how it behaves (and also reuses the same values).
  2. I increased the window size to 640x480 - test is now passing on MSEdge reliably.

@uuf6429 uuf6429 changed the title Refactor testResizeWindow; increase window size for edge Refactor testResizeWindow; increase window size for MSEdge Sep 27, 2023
Copy link
Copy Markdown
Member

@aik099 aik099 left a comment

Choose a reason for hiding this comment

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

Looks good to me. We can only see if test changes don't break anything once merged and driver tests are re-runned.

@stof stof merged commit 5bf4c47 into minkphp:master Oct 17, 2023
@uuf6429 uuf6429 deleted the improve-resize-window branch October 17, 2023 09:44
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.

3 participants