Skip to content

Comments

Wire native CSS parsing for fontVariant#55684

Open
NickGerleman wants to merge 3 commits intofacebook:mainfrom
NickGerleman:export-D94052733
Open

Wire native CSS parsing for fontVariant#55684
NickGerleman wants to merge 3 commits intofacebook:mainfrom
NickGerleman:export-D94052733

Conversation

@NickGerleman
Copy link
Contributor

Summary:
Gate processFontVariant behind enableNativeCSSParsing(). When the flag is on, CSS font-variant strings like "small-caps oldstyle-nums" are parsed natively using the existing CSS font-variant parser instead of being preprocessed in JS.

Also removes react_native_expect(false) hard error on unknown font variant values, replacing it with a graceful skip.

Changelog: [Internal]

Differential Revision: D94052733

Summary:
Gate `processTransformOrigin` behind `enableNativeCSSParsing()`. When the flag is on, CSS transform-origin strings like `"top left"` or `"10px 50%"` are parsed natively using the existing CSS transform-origin parser instead of being preprocessed in JS.

Changelog: [Internal]

Differential Revision: D94052730
Summary:
Gate `processAspectRatio` behind `enableNativeCSSParsing()`. When the flag is on, CSS ratio strings like `"16/9"` and number strings are parsed natively using the existing CSS ratio parser instead of being preprocessed in JS. The parsing is done in `fromRawValue(... FloatOptional &)` — string values are only sent for aspectRatio; other FloatOptional yoga props never receive strings from JS.

Changelog: [Internal]

Differential Revision: D94052732
@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 Feb 23, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 23, 2026

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

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Feb 23, 2026
Summary:

Gate `processFontVariant` behind `enableNativeCSSParsing()`. When the flag is on, CSS font-variant strings like `"small-caps oldstyle-nums"` are parsed natively using the existing CSS font-variant parser instead of being preprocessed in JS.

Also removes `react_native_expect(false)` hard error on unknown font variant values, replacing it with a graceful skip.

Changelog: [Internal]

Differential Revision: D94052733
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Feb 23, 2026
Summary:

Gate `processFontVariant` behind `enableNativeCSSParsing()`. When the flag is on, CSS font-variant strings like `"small-caps oldstyle-nums"` are parsed natively using the existing CSS font-variant parser instead of being preprocessed in JS.

Also removes `react_native_expect(false)` hard error on unknown font variant values, replacing it with a graceful skip.

Changelog: [Internal]

Differential Revision: D94052733
Summary:
Pull Request resolved: facebook#55684

Gate `processFontVariant` behind `enableNativeCSSParsing()`. When the flag is on, CSS font-variant strings like `"small-caps oldstyle-nums"` are parsed natively using the existing CSS font-variant parser instead of being preprocessed in JS.

Also removes `react_native_expect(false)` hard error on unknown font variant values, replacing it with a graceful skip.

Changelog: [Internal]

Differential Revision: D94052733
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. fb-exported meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants