Skip to content

feat: allowFontScaling prop#620

Open
hejsztynx wants to merge 9 commits into
mainfrom
@ksienkiewicz/feat-allow-font-scaling
Open

feat: allowFontScaling prop#620
hejsztynx wants to merge 9 commits into
mainfrom
@ksienkiewicz/feat-allow-font-scaling

Conversation

@hejsztynx
Copy link
Copy Markdown
Collaborator

@hejsztynx hejsztynx commented May 27, 2026

Summary

Thanks for the feature suggestion in this issue!

Implemented a allowFontScaling prop

  • when set to true, the font size will be scaled to satisfy the system's accessibility font size settings (default behaviour)
  • when set to false, it works independently of the current settings

Videos

EnrichedTextInput

Screen.Recording.2026-05-27.at.16.18.28.mov

EnrichedText

Screen.Recording.2026-05-27.at.16.20.20.mov

Compatibility

OS Implemented
iOS
Android

Checklist

  • E2E tests are passing
  • Required E2E tests have been added (if applicable)

@hejsztynx hejsztynx force-pushed the @ksienkiewicz/feat-allow-font-scaling branch from 086eafc to c7927aa Compare May 29, 2026 10:36
- runFlow:
file: "../subflows/capture_or_assert_screenshot.yaml"
env:
SCREENSHOT_NAME: "font_scaling__placeholder"
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.

Suggested change
SCREENSHOT_NAME: "font_scaling__placeholder"
SCREENSHOT_NAME: "font_scaling_placeholder"

@@ -0,0 +1,34 @@
appId: swmansion.enriched.example
tags:
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.

You should either mark this as iOS-only or change the HTML value, because the Android screenshot is not correct for alignment - the Android text alignment support hasn’t been merged yet.

onPasteImages={(e) => editor.handlePasteImagesEvent(e.nativeEvent)}
useHtmlNormalizer
testID="editor-input"
allowFontScaling
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.

It's redundant, as it's a default value, correct?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants