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

Fix for Android mobile browsers: soft keyboards do not send key events except for ENTER so we just take the last line from textarea without the prompt prefix + fix for iOS mobile browsers: soft keyboard did not show on some iOS versions #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

markostamcar
Copy link

Note: Still having trouble on some iOS versions where the soft keyboard won't open at all but since I don't own any iOS device, it's hard to test. Will be trying using iOS 12 demo at https://appetize.io/demo

… for ENTER so we just take the last line from textarea without the prompt prefix
@andersevenrud andersevenrud self-requested a review May 13, 2020 16:52
@markostamcar markostamcar changed the title Fix for mobile browsers: soft keyboards do not send key events except for ENTER so we just take the last line from textarea without the prompt prefix Fix for Android mobile browsers: soft keyboards do not send key events except for ENTER so we just take the last line from textarea without the prompt prefix May 13, 2020
@markostamcar
Copy link
Author

I have now commited the iOS Safari keyboard fix, too :)

@markostamcar markostamcar changed the title Fix for Android mobile browsers: soft keyboards do not send key events except for ENTER so we just take the last line from textarea without the prompt prefix Fix for Android mobile browsers: soft keyboards do not send key events except for ENTER so we just take the last line from textarea without the prompt prefix + fix for iOS mobile browsers: soft keyboard did not show on some iOS versions May 15, 2020
@andersevenrud andersevenrud self-assigned this Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants