Skip to content

Fix node ownership when display: contents is used (#1924)#1924

Closed
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:@jpiasecki/fix-contents-crash
Closed

Fix node ownership when display: contents is used (#1924)#1924
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:@jpiasecki/fix-contents-crash

Conversation

@j-piasecki

@j-piasecki j-piasecki commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

Summary:
X-link: react/react-native#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when display: contents is used in absolutely positioned subtrees

Fixes an edge case where nodes with display: contents weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki

@vercel

vercel Bot commented Apr 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
yoga-website Ready Ready Preview, Comment Apr 21, 2026 5:02am

Request Review

@meta-codesync

meta-codesync Bot commented Apr 13, 2026

Copy link
Copy Markdown

@j-piasecki has imported this pull request. If you are a Meta employee, you can view this in D100581579.

j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 13, 2026
Summary:
Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: react/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 20, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Apr 21, 2026
Summary:

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
Summary:
X-link: facebook/react-native#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.


Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki
@meta-codesync meta-codesync Bot changed the title Fix node ownership when display: contents is used Fix node ownership when display: contents is used (#1924) Apr 21, 2026
@j-piasecki j-piasecki force-pushed the @jpiasecki/fix-contents-crash branch from e77f74c to 4cb9bdd Compare April 21, 2026 05:01
@meta-codesync

meta-codesync Bot commented Apr 21, 2026

Copy link
Copy Markdown

@j-piasecki has exported this pull request. If you are a Meta employee, you can view the originating Diff in D100581579.

@meta-codesync meta-codesync Bot closed this in 966107f Apr 21, 2026
meta-codesync Bot pushed a commit to react/react-native that referenced this pull request Apr 21, 2026
Summary:
Pull Request resolved: #56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: react/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki

fbshipit-source-id: e05e9a4076bd11a71be438ef910a262044659a9b
@meta-codesync

meta-codesync Bot commented Apr 21, 2026

Copy link
Copy Markdown

@j-piasecki merged this pull request in 966107f.

j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Jun 9, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki

fbshipit-source-id: e05e9a4076bd11a71be438ef910a262044659a9b
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Jun 9, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki

fbshipit-source-id: e05e9a4076bd11a71be438ef910a262044659a9b
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Jun 9, 2026
Summary:
Pull Request resolved: facebook#56422

Changelog: [GENERAL][FIXED] Fixed Yoga node ownership when `display: contents` is used in absolutely positioned subtrees

Fixes an edge case where nodes with `display: contents` weren't cloned properly inside absolutely positioned subtrees.

Adds a test case covering this scenario.

X-link: facebook/yoga#1924

Reviewed By: NickGerleman

Differential Revision: D100581579

Pulled By: j-piasecki

fbshipit-source-id: e05e9a4076bd11a71be438ef910a262044659a9b
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.

1 participant