Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TextEditView: Soft keyboard now opens on first tap and cursor is positioned correctly. #71

Closed
wants to merge 2 commits into from

Conversation

jsdevlover
Copy link

No description provided.

@jsdevlover jsdevlover changed the title Soft keyboard now opens on first tap and cursor is positioned correctly. TextEditView: Soft keyboard now opens on first tap and cursor is positioned correctly. Nov 15, 2015
@collingreen
Copy link
Contributor

Thanks @jsdevlover. I'll try this out.

@collingreen
Copy link
Contributor

@jsdevlover I'm not seeing either problem on my test phones without your changes (s4, android 4.4.4); iphone 5, ios9.1)

Can you tell me how to reproduce these issues so I can verify your changes fix them?

@jsdevlover
Copy link
Author

@collingreen , you have already reproduced first part (opening keyboard on first click) and confirmed here (2nd bullet point in Android Summary).

For cursor position, type in some text in the TextEditView lose the focus and try refocusing TextEditView. This time the cursor is positioned before the text. It should be positioned at the end of the text.

Please let me know if these are fixed in any of the latest commits and if yes let me know the devkit-core version I should use.
Thanks.

@collingreen
Copy link
Contributor

You're right! I apparently have two textedit test apps - one shows these issues, the other does not. Digging into the differences now.

@collingreen
Copy link
Contributor

More follow ups in the posted issue. tl;dr - 3 letter color codes seem to be an issue.

@jsdevlover
Copy link
Author

Since we have identified the correct issue. I'm closing this PR.

@jsdevlover jsdevlover closed this Nov 18, 2015
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