fix(android): unify line break strategy across measuring and drawing, add line break props

[eszlamczyk]

What/Why?

Adds textBreakStrategy (Android) and lineBreakStrategyIOS (iOS) props to both EnrichedMarkdownText, mirroring the same props available on React Native's core Text component.

On Android, the break strategy must match between the measurement pass (StaticLayout.Builder) and the render pass (TextView.breakStrategy) - a mismatch causes the measured line count to differ from the rendered one, which manifests as incorrect view height and clipped or scrolled content, most visibly with long links that wrap across lines. Previously the strategy was hardcoded to simple via a top-level function; this PR promotes it to a singleton (BreakStrategyUtils) that both paths read from, so changing the prop keeps them in sync.

On iOS, the lineBreakStrategy is applied to the NSParagraphStyle used when building attributed strings, allowing consumers to opt into standard, hangul-word, or push-out
breaking on iOS 14+.

Note: the defaults were kept in a way not to have any breaking changes

Testing

Go to playground in example app

paste (via set markdown option) string containing long text/links for example:

Check out this link: [A very long link label that should wrap onto the next line in a narrow container (https://example.com)

And a bare URL: https://this-is-a-very-long-url-that-will-force-line-wrapping.example.com/some/deep/path

To make this more visible adjust max width of the container (to for example 220)

Alternative Test

Create any view that has:

• some <Text>
• Markdown Text block with text before and link that is longer than one line
• some <Text>

Example

Replace App.tsx in example app with

import { Text, View } from 'react-native';
import { EnrichedMarkdownText } from 'react-native-enriched-markdown';

export default function App() {
  return (
    <View style={{ padding: 16, marginTop: 50 }}>
      <Text>Some text before rendered Markdown</Text>
      <EnrichedMarkdownText
        markdown={
          'Find related: https://www.example.com/a-long-bit-of-text-that-keeps-going'
        }
      />
      <Text>Some text after rendered Markdown</Text>
    </View>
  );
}

Screenshots

Main test

Android

simple	highQuality (default)	balanced

iOS

none (default)	standard	hangul-word	push-out

Alternative test

Note: Before photo is partialy scrolled to display the issue

Before	After

PR Checklist

• Code compiles and runs on iOS
• Code compiles and runs on Android
• Updated documentation/README if applicable
• Ran example app to verify changes
• E2E tests are passing
• Required E2E tests have been added (if applicable)

closes #366

[hryhoriiK97]

Looks like this approach fixes the current problem, but changing from HIGH_QUALITY to SIMPLE could be a breaking change for existing library consumers. I'm thinking about introducing a lineBreakStrategy prop — the same as RN Text has: https://reactnative.dev/docs/text#textbreakstrategy-android

[Copilot]

Pull request overview

This PR adds cross-platform line-breaking configuration props to the library’s Markdown text components and updates native iOS/Android rendering/measurement to keep line breaking consistent (notably addressing Android height mis-measurement/clipping for long wrapped URLs/links).

Changes:

• Add textBreakStrategy (Android) and lineBreakStrategyIOS (iOS) props to the public TS props and native codegen props for EnrichedMarkdownText / EnrichedMarkdown.
• iOS: apply lineBreakStrategyIOS via NSParagraphStyle during attributed-string building.
• Android: centralize break-strategy resolution and apply it in both StaticLayout.Builder (measurement) and TextView.breakStrategy (render).

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/types/MarkdownTextProps.ts	Exposes new break-strategy props on EnrichedMarkdownTextProps.
src/native/EnrichedMarkdownText.tsx	Wires new props from JS wrapper into native component props.
src/EnrichedMarkdownTextNativeComponent.ts	Adds codegen native prop definitions for EnrichedMarkdownText.
src/EnrichedMarkdownNativeComponent.ts	Adds codegen native prop definitions for EnrichedMarkdown (GFM/container renderer).
ios/utils/ParagraphStyleUtils.m	Introduces global line-break strategy and applies it to paragraph styles.
ios/utils/ParagraphStyleUtils.h	Declares the new line-break strategy setter.
ios/EnrichedMarkdownText.mm	Plumbs lineBreakStrategyIOS prop changes into the iOS text renderer.
ios/EnrichedMarkdown.mm	Plumbs lineBreakStrategyIOS prop changes into the iOS GFM/container renderer.
android/.../TextViewSetup.kt	Applies resolved break strategy to the rendered TextView on API 23+.
android/.../utils/common/BreakStrategyUtils.kt	Adds a shared resolver for Android break-strategy selection.
android/.../MeasurementStore.kt	Applies the resolved break strategy during StaticLayout measurement on API 23+.
android/.../EnrichedMarkdownTextManager.kt	Exposes textBreakStrategy as a React prop (and adds iOS-only prop no-ops).
android/.../EnrichedMarkdownText.kt	Implements setTextBreakStrategy and invalidates measurement/re-render.
android/.../EnrichedMarkdownManager.kt	Exposes textBreakStrategy as a React prop (and adds iOS-only prop no-ops).
android/.../EnrichedMarkdown.kt	Propagates break-strategy changes to segment TextViews and forces height recalculation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[hryhoriiK97]

The default/fallback here is still "simple", but managers, codegen, and the JS wrapper all default to "highQuality". Let's use 'highQuality' as default.

[hryhoriiK97]

BreakStrategyUtils is a process-wide singleton, but textBreakStrategy is a per-view prop - with multiple markdown views on screen, the last setter wins for all measurement/render paths.

RN Text doesn't do this: each shadow node stores its own strategy (mTextBreakStrategy / ParagraphAttributes.textBreakStrategy) and passes the same value to both StaticLayout.Builder (measure) and ReactTextView.setBreakStrategy() (render).

Could we follow the existing MeasurementStore.fontScalingSettings pattern instead - store per viewId, resolve during measurement, set on each view's TextView?

[hryhoriiK97]

This file already has statics like kBlockSpacerTemplate, but those are immutable shared templates (copied per use, same for all views).

gLineBreakStrategy is different — it's mutable state for a per-view prop. If two markdown views use different lineBreakStrategyIOS values, the last setter wins in getOrCreateParagraphStyle.

Font scaling is already stored per view here; RN Text does the same for lineBreakStrategy. Could we store this on the view/renderer and pass it through instead of a global?

[hryhoriiK97]

Could we update API_RENCE doc with these props? 🙏