Skip to content

Scroll SectionList viewability test past clipped bounds - #59572

Open
javache wants to merge 1 commit into
react:mainfrom
javache:export-D128053674
Open

javache wants to merge 1 commit into
react:mainfrom
javache:export-D128053674

Conversation

@javache

@javache javache commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Summary:
Scroll the horizontal viewability flow to Brownie before asserting that Cheesecake appears in the callback output. Android accessibility bounds are clipped to the viewport, so Maestro can report 100% visibility while Cheesecake is still partly off screen. Revealing the following item brings Cheesecake fully inside the viewport without weakening the viewability assertion or changing its timeout.

Changelog: [Internal]

Differential Revision: D128053674

Summary:
Scroll the horizontal viewability flow to Brownie before asserting that Cheesecake appears in the callback output. Android accessibility bounds are clipped to the viewport, so Maestro can report 100% visibility while Cheesecake is still partly off screen. Revealing the following item brings Cheesecake fully inside the viewport without weakening the viewability assertion or changing its timeout.

Changelog: [Internal]

Differential Revision: D128053674
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 28, 2026
@meta-codesync

meta-codesync Bot commented Oct 28, 2026

Copy link
Copy Markdown

@javache has exported this pull request. If you are a Meta employee, you can view the originating Diff in D128053674.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant