-
Notifications
You must be signed in to change notification settings - Fork 24.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: maxFontSizeMultiplier is not being respected #47499
Comments
Is this working well if you turn off the New Architecture? Asking to understand if this is a Fabric regression or not |
@cortinico It is working properly on the legacy architecture in 0.75.4. |
And how about legacy arch on 0.76? |
Yes, I can confirm it is working on 0.76.1 with the legacy arch. |
I've added support (RickardZrinski@140d9eb) and hoping to open a pull request tomorrow. Just need to do some more manual testing / compare with the old architecture and do some screenshots for the PR. Edit: still working on it :) |
|
Everything is working on Android and iOS, except that there is some weirdness happening on the new architecture with reapplying changes made in Settings when switching back to the app (e.g. you've changed font size in accessibility settings but you need to restart the app for it to apply). I don't think this issue is specific to my PR but rather a general issue in the new architecture. Though I'll need to verify that to be sure. Edit: yeah, the reapplying of settings is a general issue on the new architecture. On iOS you can switch back and forth to your app after changing Settings and usually after the second roundtrip it gets applied. On Android it does apply after returning from Settings, but line height (seemingly) is messed up until you restart the app. I should probably open an issue on this... |
Description
The maxFontSizeMultiplier prop is not being respected for Text and TextInput components. When accessibility settings are changed to increase the display text, the font size increases beyond the multiplier set.
Steps to reproduce
React Native Version
0.76.1
Affected Platforms
Runtime - Android, Runtime - iOS
Areas
Fabric - The New Renderer
Output of
npx react-native info
Stacktrace or Logs
Reproducer
https://github.com/mysport12/maxFontSizeMultiplierBug
Screenshots and Videos
The text was updated successfully, but these errors were encountered: