Skip to content

Conversation

@sammy-SC
Copy link
Contributor

Summary

To enable Concurrent Features in React Native, we need to pass flag to render surface with concurrent root.

Test Plan

Tested separately with React Native.

@sizebot
Copy link

sizebot commented May 24, 2021

Comparing: a731a51...46c279c

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 = 126.00 kB 126.00 kB = 40.41 kB 40.41 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 128.82 kB 128.82 kB = 41.35 kB 41.35 kB
facebook-www/ReactDOM-prod.classic.js = 406.09 kB 406.09 kB = 75.11 kB 75.11 kB
facebook-www/ReactDOM-prod.modern.js = 394.46 kB 394.46 kB = 73.29 kB 73.29 kB
facebook-www/ReactDOMForked-prod.classic.js = 406.09 kB 406.09 kB = 75.11 kB 75.11 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 46c279c

@sammy-SC sammy-SC force-pushed the rn-concurrent-root branch from 83e6cd3 to a345266 Compare May 24, 2021 09:32
@sammy-SC sammy-SC force-pushed the rn-concurrent-root branch from a345266 to aa337a9 Compare May 24, 2021 09:32
Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you considered mirroring the new createRoot API instead, like we do in DOM? It would make sense to me if those were aligned, unless there was some specific conversation about not doing that.

@rickhanlonii
Copy link
Member

Nice!

@rickhanlonii rickhanlonii merged commit 459c34f into facebook:master May 24, 2021
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request May 25, 2021
Summary:
This sync includes the following changes:
- **[459c34fde](facebook/react@459c34fde )**: Add flag concurrent root ([#21552](facebook/react#21552)) //<Samuel Susla>//

Changelog:
[General][Changed] - React Native sync for revisions 3169430...459c34f

jest_e2e[run_all_tests]

Reviewed By: mdvacca

Differential Revision: D28641463

fbshipit-source-id: 0930a8f82e49b222582995e4c8b2c8a41564bf53
@sammy-SC sammy-SC deleted the rn-concurrent-root branch June 9, 2021 11:29
koto pushed a commit to koto/react that referenced this pull request Jun 15, 2021
* Add flag concurrent root to Fabric render function

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants