Skip to content

Conversation

@acdlite
Copy link
Collaborator

@acdlite acdlite commented Nov 7, 2025

If an error is thrown inside a hidden Activity, it should not escape into the visible part of the UI. Conceptually, a hidden Activity boundary is not part of the current UI; it's the same as an unmounted tree, except for the fact that the state will be restored if it's later revealed.

Fixes:

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

react-sizebot commented Nov 7, 2025

Comparing: a10ff9c...bdbc30a

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 +0.02% 607.97 kB 608.07 kB +0.02% 107.66 kB 107.69 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 +0.02% 665.88 kB 665.99 kB +0.01% 117.37 kB 117.39 kB
facebook-www/ReactDOM-prod.classic.js +0.02% 693.25 kB 693.36 kB +0.01% 122.00 kB 122.02 kB
facebook-www/ReactDOM-prod.modern.js +0.02% 683.67 kB 683.78 kB +0.01% 120.38 kB 120.40 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against bdbc30a

Copy link
Member

@rickhanlonii rickhanlonii left a comment

Choose a reason for hiding this comment

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

Nice

If an error is thrown inside a hidden Activity, it should not escape
into the visible part of the UI. Conceptually, a hidden Activity
boundary is not part of the current UI; it's the same as an unmounted
tree, except for the fact that the state will be restored if it's
later revealed.
@acdlite acdlite force-pushed the activity-error-handling branch from b50253c to bdbc30a Compare November 7, 2025 23:12
@acdlite acdlite merged commit 717e708 into facebook:main Nov 7, 2025
239 of 240 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 7, 2025
If an error is thrown inside a hidden Activity, it should not escape
into the visible part of the UI. Conceptually, a hidden Activity
boundary is not part of the current UI; it's the same as an unmounted
tree, except for the fact that the state will be restored if it's later
revealed.

Fixes:
- #35073

DiffTrain build for [717e708](717e708)
github-actions bot pushed a commit that referenced this pull request Nov 7, 2025
If an error is thrown inside a hidden Activity, it should not escape
into the visible part of the UI. Conceptually, a hidden Activity
boundary is not part of the current UI; it's the same as an unmounted
tree, except for the fact that the state will be restored if it's later
revealed.

Fixes:
- #35073

DiffTrain build for [717e708](717e708)
github-actions bot pushed a commit to code/lib-react that referenced this pull request Nov 9, 2025
If an error is thrown inside a hidden Activity, it should not escape
into the visible part of the UI. Conceptually, a hidden Activity
boundary is not part of the current UI; it's the same as an unmounted
tree, except for the fact that the state will be restored if it's later
revealed.

Fixes:
- facebook#35073

DiffTrain build for [717e708](facebook@717e708)
github-actions bot pushed a commit to code/lib-react that referenced this pull request Nov 9, 2025
If an error is thrown inside a hidden Activity, it should not escape
into the visible part of the UI. Conceptually, a hidden Activity
boundary is not part of the current UI; it's the same as an unmounted
tree, except for the fact that the state will be restored if it's later
revealed.

Fixes:
- facebook#35073

DiffTrain build for [717e708](facebook@717e708)
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