Skip to content

Conversation

@rickhanlonii
Copy link
Member

I copied some tests from Activity-test.js and made them portal specific just to confirm my understanding of how Portals + Activity interact is correct. Seems good to include them.

@meta-cla meta-cla bot added the CLA Signed label Nov 10, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Nov 10, 2025
@react-sizebot
Copy link

Comparing: be48396...91d2e8a

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.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 608.16 kB 608.16 kB = 107.65 kB 107.65 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 666.18 kB 666.18 kB = 117.35 kB 117.35 kB
facebook-www/ReactDOM-prod.classic.js = 693.31 kB 693.31 kB = 121.97 kB 121.97 kB
facebook-www/ReactDOM-prod.modern.js = 683.73 kB 683.73 kB = 120.36 kB 120.36 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 91d2e8a

@rickhanlonii rickhanlonii merged commit 04ee54c into facebook:main Nov 11, 2025
246 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 11, 2025
I copied some tests from
[`Activity-test.js`](https://github.com/facebook/react/blob/1d68bce19c9409ed70604d1d16b70b68ce71dc4a/packages/react-reconciler/src/__tests__/Activity-test.js)
and made them portal specific just to confirm my understanding of how
Portals + Activity interact is correct. Seems good to include them.

DiffTrain build for [04ee54c](04ee54c)
manNomi pushed a commit to manNomi/react that referenced this pull request Nov 15, 2025
I copied some tests from
[`Activity-test.js`](https://github.com/facebook/react/blob/1d68bce19c9409ed70604d1d16b70b68ce71dc4a/packages/react-reconciler/src/__tests__/Activity-test.js)
and made them portal specific just to confirm my understanding of how
Portals + Activity interact is correct. Seems good to include them.
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.

3 participants