Skip to content

Wait for images before box-shadow screenshots (#57922) - #57922

Closed
cipolleschi wants to merge 3 commits into
react:mainfrom
cipolleschi:export-D115740465
Closed

Wait for images before box-shadow screenshots (#57922)#57922
cipolleschi wants to merge 3 commits into
react:mainfrom
cipolleschi:export-D115740465

Conversation

@cipolleschi

@cipolleschi cipolleschi commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary:

Changelog: [Internal]

Expose the box-shadow example test identifier only after all three images have loaded. The E2E navigation helper now waits for stable image content before capturing the screenshot, and the Nougat golden reflects the complete example.

Reviewed By: Abbondanzo

Differential Revision: D115740465

@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 Aug 12, 2026
@meta-codesync

meta-codesync Bot commented Aug 12, 2026

Copy link
Copy Markdown

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

@meta-codesync meta-codesync Bot changed the title Wait for images before box-shadow screenshots Wait for images before box-shadow screenshots (#57922) Aug 12, 2026
cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Aug 12, 2026
Summary:

Changelog: [Internal]

Expose the box-shadow example test identifier only after all three images have loaded. The E2E navigation helper now waits for stable image content before capturing the screenshot, and the Nougat golden reflects the complete example.

Differential Revision: D115740465
Summary:

Changelog: [Internal]

Exercise JavaScript briefly before ending each trace. This gives the sampling profiler a bounded opportunity to record a stack while preserving the test comparison between disabled and enabled categories.

Reviewed By: Abbondanzo

Differential Revision: D115740464
Summary:
Changelog: [General][Fixed] - Ignore stale viewability updates while enforcing minimum view time

Minimum view-time callbacks can outlive a newer viewport update. During initial layout, callbacks for intermediate viewport snapshots can fire after the current snapshot and replace the correct visible-item set. Discard callbacks whose captured visible-index set is no longer current, and cover the transition with a unit test.

Reviewed By: Abbondanzo

Differential Revision: D115740467
Summary:

Changelog: [Internal]

Expose the box-shadow example test identifier only after all three images have loaded. The E2E navigation helper now waits for stable image content before capturing the screenshot, and the Nougat golden reflects the complete example.

Reviewed By: Abbondanzo

Differential Revision: D115740465
@meta-codesync meta-codesync Bot closed this in e334b93 Aug 13, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Aug 13, 2026
@meta-codesync

meta-codesync Bot commented Aug 13, 2026

Copy link
Copy Markdown

This pull request has been merged in e334b93.

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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant