Remove outdated multiline single-side border caveat from TextInput docs - #5200
Conversation
|
Hey @theprantadutta, thanks for picking it up and providing a bit more context. I was able to verify that the border styles are applied correctly to multiline text inputs on hardware device for both platforms using Expo SDK 54, so React Native 0.81, using this Snack example: Can you also update all versioned docs, starting from 0.81 onwards? The doc files are located here: |
|
Thanks for verifying it on a device — that's the part I couldn't do myself, so I appreciate you closing that gap. Done: the caveat is now removed from One note on the history of this branch: my fork was a few commits behind and |
Simek
left a comment
There was a problem hiding this comment.
Thanks for the follow up updates and the context! 👍
Fixes #5151
The TextInput page states:
This removes that sentence. Two separate problems with it:
1. The workaround it promises is never shown. The sentence ends in a colon introducing the example below it, but that example is a plain multiline
TextInputstyled withborderColor/borderWidth— an all-sides border, applied directly to theTextInput, with no wrappingViewanywhere. So the text promises a demonstration the example does not contain, regardless of whether the limitation itself is still real. This part needs no runtime check to confirm; it's visible in the page as it stands.2. The limitation looks outdated. It came in with the original "Known issues" section in #649 (December 2018), well before the New Architecture reworked border rendering. On Android today,
ReactEditText— the view backingTextInput, multiline included — applies borders through the same per-edge path as any other view:That's
BackgroundStyleApplicator.setBorderWidth, taking aLogicalEdge, and there is no multiline special-casing in the applicator. The reporter of #5151 also tested single-side borders on a multiline input in Snack across Android, iOS, and Web and found they applied as expected.What I could not verify
I want to be straight about this rather than imply more than I did: I did not run a Snack myself. I don't have an Android SDK on this machine, so points above are source-level reading plus the reporter's testing, not my own end-to-end run on a current release.
If you'd like empirical confirmation before merging, that's completely fair — either the reporter can share their Snack, or I'm happy to have this sit until someone can check it on a device. Problem 1 stands on its own either way, so if you'd rather keep a corrected version of the limitation note, I can instead reword it and add an example that actually demonstrates the wrapping-
Viewworkaround. Just say which you prefer.I've edited only
docs/textinput.mdand left the versioned copies underwebsite/versioned_docs/alone, since those describe released versions.For disclosure: I use an AI assistant in my work, and I review and verify everything before it goes out.