Skip to content

fix: wire up enableFocusRing prop for TextInput#2870

Draft
Saadnajmi wants to merge 1 commit intomicrosoft:mainfrom
Saadnajmi:fix-enable-focus-ring-textinput
Draft

fix: wire up enableFocusRing prop for TextInput#2870
Saadnajmi wants to merge 1 commit intomicrosoft:mainfrom
Saadnajmi:fix-enable-focus-ring-textinput

Conversation

@Saadnajmi
Copy link
Collaborator

Summary

  • Forward enableFocusRing prop to the native TextInput view — it was missing from the view config and view manager

Test plan

  • Set enableFocusRing={false} on a <TextInput> and verify no focus ring appears on focus
  • Verify default behavior (focus ring shown) is unchanged

🤖 Generated with Claude Code

@changeset-bot
Copy link

changeset-bot bot commented Mar 23, 2026

⚠️ No Changeset found

Latest commit: 4d9e6ad

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Saadnajmi Saadnajmi force-pushed the fix-enable-focus-ring-textinput branch from 99b73a8 to 6b46e97 Compare March 23, 2026 06:45
Forward enableFocusRing prop to the native TextInput view — it was
missing from the view manager.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Saadnajmi Saadnajmi force-pushed the fix-enable-focus-ring-textinput branch from 2a64ffd to 4d9e6ad Compare March 23, 2026 16:07
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.

1 participant