Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Android] Fix entry cursor jump #6736

Closed
wants to merge 1 commit into from
Closed

[Android] Fix entry cursor jump #6736

wants to merge 1 commit into from

Conversation

kvpt
Copy link
Contributor

@kvpt kvpt commented Jul 1, 2019

Description of Change

It seems that under certains circonstances, native android control IsFocused state can be out of sync.

image

This cause the SetSelection to not run and the SelectionStart to be equal to 0 after text selection.
As a result the cursor jump.

Issues Resolved

API Changes

None

Platforms Affected

  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

Before
before

After
after

Testing Procedure

See #3923

PR Checklist

  • Has automated tests
  • Rebased on top of the target branch at time of PR
  • Changes adhere to coding standard

@samhouts samhouts added feedback-ticket Issue originates from https://developercommunity.visualstudio.com inactive Issue is older than 6 months and needs to be retested t/bug 🐛 labels Jul 2, 2019
@kvpt
Copy link
Contributor Author

kvpt commented Jul 2, 2019

The issue of the out of sync focus is resolved by #4730, that in return fix the problem.
So I close this.

@kvpt kvpt closed this Jul 2, 2019
@samhouts samhouts added i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often partner/cat 😻 labels Jul 3, 2019
@kvpt kvpt deleted the fix-3923 branch August 30, 2019 23:32
@samhouts samhouts added the in-progress This issue has an associated pull request that may resolve it! label Oct 2, 2019
@samhouts samhouts added this to the 2.5.0 milestone Oct 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/entry feedback-ticket Issue originates from https://developercommunity.visualstudio.com i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often in-progress This issue has an associated pull request that may resolve it! inactive Issue is older than 6 months and needs to be retested partner/cat 😻 t/bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants