Skip to content

Tests: Fix the "outside view position" test in Headless Chrome (3.x)#5729

Merged
mgol merged 1 commit intojquery:3.x-stablefrom
mgol:3.x-test-outside-view-position-fix
Nov 24, 2025
Merged

Tests: Fix the "outside view position" test in Headless Chrome (3.x)#5729
mgol merged 1 commit intojquery:3.x-stablefrom
mgol:3.x-test-outside-view-position-fix

Conversation

@mgol
Copy link
Member

@mgol mgol commented Nov 20, 2025

Summary

In Headless Chrome, when run locally on macOS together with a few other modules, the scroll handler for the appended element in the "outside view position" dimensions test doesn't fire, timing out the test & failing it as a result.

While the scroll handler may not fire, the new position data is available immediately, so just do the checks directly, without relying on scroll handlers.

main version: gh-5728

Ref gh-5728

Checklist

In Headless Chrome, when run locally on macOS together with a few other modules,
the scroll handler for the appended element in the "outside view position"
dimensions test doesn't fire, timing out the test & failing it as a result.

While the scroll handler may not fire, the new position data is available
immediately, so just do the checks directly, without relying on scroll handlers.
@mgol mgol self-assigned this Nov 20, 2025
@mgol mgol added Needs review Tests Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. labels Nov 20, 2025
@mgol mgol added this to the 3.8.0 milestone Nov 20, 2025
@timmywil timmywil removed Needs review Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. labels Nov 24, 2025
@mgol
Copy link
Member Author

mgol commented Nov 24, 2025

I tested it on my fork and BrowserStack was green.

mgol added a commit that referenced this pull request Nov 24, 2025
In Headless Chrome, when run locally on macOS together with a few other modules,
the scroll handler for the appended element in the "outside view position"
dimensions test doesn't fire, timing out the test & failing it as a result.

While the scroll handler may not fire, the new position data is available
immediately, so just do the checks directly, without relying on scroll handlers.

Closes gh-5728
Ref gh-5729
@mgol mgol merged commit c4c9090 into jquery:3.x-stable Nov 24, 2025
16 checks passed
@mgol mgol deleted the 3.x-test-outside-view-position-fix branch November 24, 2025 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants