Skip to content

feat(ios): bring the example Text screen to RN parity - #697

Open
maksg wants to merge 2 commits into
ios-gfm/01-task-listsfrom
feat/ios-example-text-screen
Open

feat(ios): bring the example Text screen to RN parity#697
maksg wants to merge 2 commits into
ios-gfm/01-task-listsfrom
feat/ios-example-text-screen

Conversation

@maksg

@maksg maksg commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

What/Why?

Testing

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)

maksg and others added 2 commits August 21, 2026 16:30
Port the react-native-example TextScreen behaviors that the native
package supports: superscript/subscript md4c flags, a link-press alert
with Open in Browser/Cancel instead of opening links directly, the RN
selection color (#5A52FA), and a TaskList theme block matching the RN
example's taskList style so the new checkboxes render with the same
colors and geometry. Also fixes literal backslash-dollar escapes that
were copied verbatim from the RN sample's template literal into the
sample document's JS code block.

Custom context menu items and display math are not ported: the package
has no custom selection-menu API and no math renderer yet.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…es it

AppShell loaded sample_markdown.md in onAppear, but the
navigationDestination closure could build TextScreen from a stale view
value captured while the state was still empty, leaving the Text screen
permanently blank (deterministic on the iOS 27 simulator, intermittent
on 26.5). Initialize the state from the bundle directly so every
destination sees the loaded document.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

2 participants