Skip to content

RN: Overflow.fromString accepts null and returns non-null (#57638)#57638

Closed
twasilczyk wants to merge 1 commit into
react:mainfrom
twasilczyk:export-D113186028
Closed

RN: Overflow.fromString accepts null and returns non-null (#57638)#57638
twasilczyk wants to merge 1 commit into
react:mainfrom
twasilczyk:export-D113186028

Conversation

@twasilczyk

@twasilczyk twasilczyk commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary:

Simplifies current and future uses of overflow fields.

ReactViewGroup _overflow field is no longer nullable - it's default is now VISIBLE instead of null. The never-set or unrecognized case is no longer detectable, but it's fine - it will just default to VISIBLE, as all uses were already doing so.

Differential Revision: D113186028

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 22, 2026
@meta-codesync

meta-codesync Bot commented Jul 22, 2026

Copy link
Copy Markdown

@twasilczyk has exported this pull request. If you are a Meta employee, you can view the originating Diff in D113186028.

Summary:
Pull Request resolved: react#57638

Simplifies current and future uses of overflow fields.

ReactViewGroup _overflow field is no longer nullable - it's default is now VISIBLE instead of null. The never-set or unrecognized case is no longer detectable, but it's fine - it will just default to VISIBLE, as all uses were already doing so.

Differential Revision: D113186028
@meta-codesync meta-codesync Bot changed the title RN: Overflow.fromString accepts null and returns non-null RN: Overflow.fromString accepts null and returns non-null (#57638) Jul 23, 2026
@meta-codesync meta-codesync Bot closed this in 50a35db Jul 23, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Jul 23, 2026
@meta-codesync

meta-codesync Bot commented Jul 23, 2026

Copy link
Copy Markdown

This pull request has been merged in 50a35db.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant