Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DOM] Apply Fragment listeners to children inserted into portals later - #37166

Merged
jackpope merged 2 commits into
mainfrom
fragment-refs/portal-child-event-listeners
Aug 12, 2026
Merged

[DOM] Apply Fragment listeners to children inserted into portals later#37166
jackpope merged 2 commits into
mainfrom
fragment-refs/portal-child-event-listeners

Conversation

@jackpope

@jackpope jackpope commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Commit-time fragment ancestry stopped at HostPortal via isHostParent, so a child added after listeners were registered never received them even though existing portal siblings did. Collect fragment parents past portals and other non-HostComponent host parents so commit bookkeeping matches HostComponent/HostRoot fragment ancestry.

@meta-cla meta-cla Bot added the CLA Signed label Jul 31, 2026
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from d109b58 to e8474b6 Compare July 31, 2026 19:14
@github-actions github-actions Bot added the React Core Team Opened by a member of the React Core Team label Jul 31, 2026
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from e8474b6 to c91f719 Compare July 31, 2026 20:15
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from c91f719 to 96726e3 Compare July 31, 2026 20:55
@jackpope
jackpope requested a review from eps1lon July 31, 2026 21:08
@jackpope
jackpope marked this pull request as ready for review July 31, 2026 21:09
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from 96726e3 to 4c2ca3b Compare August 9, 2026 19:15
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from 4c2ca3b to 185f5cf Compare August 11, 2026 03:49
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from 185f5cf to fd3f42a Compare August 12, 2026 00:50
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from fd3f42a to b29987a Compare August 12, 2026 01:05

@poteto poteto left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM (light skim). Stacking stamps for the fragment-refs land sequence.

@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from b29987a to 9851ac0 Compare August 12, 2026 01:13
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from 9851ac0 to 89bb7fb Compare August 12, 2026 01:19
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from 89bb7fb to 239f6dc Compare August 12, 2026 01:25
Base automatically changed from fragment-refs/dispatch-event-document-container to main August 12, 2026 01:32
Commit-time fragment ancestry stopped at HostPortal via isHostParent, so
a child added after listeners were registered never received them even
though existing portal siblings did. Collect fragment parents past
portals and other non-HostComponent host parents so commit bookkeeping
matches HostComponent/HostRoot fragment ancestry.
@jackpope
jackpope force-pushed the fragment-refs/portal-child-event-listeners branch from 239f6dc to 24f3163 Compare August 12, 2026 01:32
@react-sizebot

Copy link
Copy Markdown

Comparing: 305feb9...24f3163

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 = 7.35 kB 7.35 kB = 1.96 kB 1.96 kB
oss-stable/react-dom/cjs/react-dom-client.production.js +0.01% 620.70 kB 620.77 kB +0.02% 109.73 kB 109.75 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 7.36 kB 7.36 kB = 1.96 kB 1.96 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 691.91 kB 691.98 kB +0.02% 121.22 kB 121.25 kB
facebook-www/ReactDOM-prod.classic.js = 712.11 kB 712.17 kB +0.03% 124.69 kB 124.73 kB
facebook-www/ReactDOM-prod.modern.js = 702.42 kB 702.49 kB +0.03% 123.11 kB 123.15 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 24f3163

@jackpope
jackpope merged commit fdaa617 into main Aug 12, 2026
238 checks passed
github-actions Bot pushed a commit that referenced this pull request Aug 12, 2026
#37166)

Commit-time fragment ancestry stopped at HostPortal via isHostParent, so
a child added after listeners were registered never received them even
though existing portal siblings did. Collect fragment parents past
portals and other non-HostComponent host parents so commit bookkeeping
matches HostComponent/HostRoot fragment ancestry.

DiffTrain build for [fdaa617](fdaa617)
github-actions Bot pushed a commit that referenced this pull request Aug 12, 2026
#37166)

Commit-time fragment ancestry stopped at HostPortal via isHostParent, so
a child added after listeners were registered never received them even
though existing portal siblings did. Collect fragment parents past
portals and other non-HostComponent host parents so commit bookkeeping
matches HostComponent/HostRoot fragment ancestry.

DiffTrain build for [fdaa617](fdaa617)
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