Skip to content

Conversation

@rectified95
Copy link
Contributor

@rectified95 rectified95 commented Apr 23, 2021

  • Change TextInput cursor behavior

As stated in issue #7283, the cursor position changes seem to be
somewhat different on each platform. Since we're already adding code in
react-native-windows to change the Windows default cursor positioning
behavior, we might as well make it a bit more consistent with other
platforms. The change here matches the behavior on iOS, macOS and Web
(so it's only different from Android, which seems to keep cursor
position relative to the TextInput value length constant).

Fixes #7283

  • Change files

Closes #7608

Microsoft Reviewers: Open in CodeFlow

* Change TextInput cursor behavior

As stated in issue microsoft#7283, the cursor position changes seem to be
somewhat different on each platform. Since we're already adding code in
react-native-windows to change the Windows default cursor positioning
behavior, we might as well make it a bit more consistent with other
platforms. The change here matches the behavior on iOS, macOS and Web
(so it's only different from Android, which seems to keep cursor
position relative to the TextInput value length constant).

Fixes microsoft#7283

* Change files
@rectified95 rectified95 requested a review from a team as a code owner April 23, 2021 00:39
@rectified95 rectified95 added the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Apr 23, 2021
@ghost
Copy link

ghost commented Apr 23, 2021

Hello @rectified95!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 10 hours. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 51feac9 into microsoft:0.64-stable Apr 23, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: TextInput AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TextInput cursor placement still not correct on 0.64 when modifying value prop?

3 participants