Skip to content

fix: Take permutations screenshot before restoring window height#214

Merged
amanabiy merged 1 commit into
mainfrom
fix/capture-permutations-before-restoring-height
May 19, 2026
Merged

fix: Take permutations screenshot before restoring window height#214
amanabiy merged 1 commit into
mainfrom
fix/capture-permutations-before-restoring-height

Conversation

@jperals
Copy link
Copy Markdown
Member

@jperals jperals commented May 19, 2026

Restore the window size after taking the permutation screenshots, not before —otherwise scrollbars can unexpectedly shift the content for the screenshots and cause a mismatch with the previously taken measurements.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jperals jperals changed the title fix: Take permutations screenshot before restoring widnow height fix: Take permutations screenshot before restoring window height May 19, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.19%. Comparing base (1c6161c) to head (6fb41f6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #214   +/-   ##
=======================================
  Coverage   92.19%   92.19%           
=======================================
  Files          18       18           
  Lines         423      423           
  Branches       69       69           
=======================================
  Hits          390      390           
  Misses         29       29           
  Partials        4        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jperals jperals marked this pull request as ready for review May 19, 2026 11:46
@jperals jperals requested a review from a team as a code owner May 19, 2026 11:46
@jperals jperals requested review from pan-kot and removed request for a team May 19, 2026 11:46
@amanabiy amanabiy added this pull request to the merge queue May 19, 2026
Merged via the queue into main with commit 58e14ca May 19, 2026
45 checks passed
@amanabiy amanabiy deleted the fix/capture-permutations-before-restoring-height branch May 19, 2026 15:14
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