Skip to content

[Transition Tracing] Push Transition When Offscreen Becomes Visible#24718

Merged
lunaruan merged 2 commits into
react:mainfrom
lunaruan:push_transitions
Jun 15, 2022
Merged

[Transition Tracing] Push Transition When Offscreen Becomes Visible#24718
lunaruan merged 2 commits into
react:mainfrom
lunaruan:push_transitions

Conversation

@lunaruan

Copy link
Copy Markdown
Contributor

This PR pushes all of a suspense boundary's transitions onto the transition stack when it goes from hidden to visible so we can pass it to any child suspense boundaries or tracing markers.

@lunaruan
lunaruan requested a review from acdlite June 14, 2022 00:57
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Jun 14, 2022
@sizebot

sizebot commented Jun 14, 2022

Copy link
Copy Markdown

Comparing: be229c5...7da489b

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 131.76 kB 131.76 kB = 42.31 kB 42.31 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 137.03 kB 137.03 kB = 43.90 kB 43.90 kB
facebook-www/ReactDOM-prod.classic.js = 440.91 kB 440.91 kB = 80.63 kB 80.63 kB
facebook-www/ReactDOM-prod.modern.js = 426.21 kB 426.21 kB = 78.44 kB 78.44 kB
facebook-www/ReactDOMForked-prod.classic.js = 441.61 kB 441.61 kB = 80.85 kB 80.85 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 7da489b

@lunaruan
lunaruan merged commit fcd720d into react:main Jun 15, 2022
facebook-github-bot pushed a commit to react/react-native that referenced this pull request Aug 17, 2022
Summary:
This sync includes the following changes:
- **[229c86a](react/react@229c86af0 )**: Revert "Land enableClientRenderFallbackOnTextMismatch" ([#24738](react/react#24738)) //<Andrew Clark>//
- **[c3d7a7e](react/react@c3d7a7e3d )**: Bugfix: Offscreen instance is null during setState ([#24734](react/react#24734)) //<Andrew Clark>//
- **[fcd720d](react/react@fcd720d36 )**: [Transition Tracing] Push Transition When Offscreen Becomes Visible ([#24718](react/react#24718)) //<Luna Ruan>//
- **[5cc2487](react/react@5cc2487e0 )**: bump versions for next release ([#24725](react/react#24725)) //<Josh Story>//
- **[54f17e4](react/react@54f17e490 )**: [Transition Tracing] Fix Cache and Transitions Pop Order ([#24719](react/react#24719)) //<Luna Ruan>//
- **[7cf8dfd](react/react@7cf8dfd94 )**: [Transition Tracing] Create/Process Marker Complete Callback ([#24700](react/react#24700)) //<Luna Ruan>//
- **[327e4a1](react/react@327e4a1f9 )**: [Follow-up] Land enableClientRenderFallbackOnTextMismatch //<Andrew Clark>//

Changelog:
[General][Changed] - React Native sync for revisions a8c9cb1...229c86a

jest_e2e[run_all_tests]

Reviewed By: rickhanlonii

Differential Revision: D38738652

fbshipit-source-id: 35b6b3cbfdbdafc28a356b53af6456aaa1949432
roryabraham pushed a commit to Expensify/react-native that referenced this pull request Aug 17, 2022
Summary:
This sync includes the following changes:
- **[229c86a](react/react@229c86af0 )**: Revert "Land enableClientRenderFallbackOnTextMismatch" ([react#24738](react/react#24738)) //<Andrew Clark>//
- **[c3d7a7e](react/react@c3d7a7e3d )**: Bugfix: Offscreen instance is null during setState ([react#24734](react/react#24734)) //<Andrew Clark>//
- **[fcd720d](react/react@fcd720d36 )**: [Transition Tracing] Push Transition When Offscreen Becomes Visible ([react#24718](react/react#24718)) //<Luna Ruan>//
- **[5cc2487](react/react@5cc2487e0 )**: bump versions for next release ([react#24725](react/react#24725)) //<Josh Story>//
- **[54f17e4](react/react@54f17e490 )**: [Transition Tracing] Fix Cache and Transitions Pop Order ([react#24719](react/react#24719)) //<Luna Ruan>//
- **[7cf8dfd](react/react@7cf8dfd94 )**: [Transition Tracing] Create/Process Marker Complete Callback ([react#24700](react/react#24700)) //<Luna Ruan>//
- **[327e4a1](react/react@327e4a1f9 )**: [Follow-up] Land enableClientRenderFallbackOnTextMismatch //<Andrew Clark>//

Changelog:
[General][Changed] - React Native sync for revisions a8c9cb1...229c86a

jest_e2e[run_all_tests]

Reviewed By: rickhanlonii

Differential Revision: D38738652

fbshipit-source-id: 35b6b3cbfdbdafc28a356b53af6456aaa1949432
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
…eact#24718)

This PR pushes all of a suspense boundary's transitions onto the transition stack when it goes from hidden to visible so we can pass it to any child suspense boundaries or tracing markers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants