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

[Android] Fix Gboard enter detection in EnterPressedWatcher #3590

Merged
merged 6 commits into from
Jun 16, 2021

Conversation

AmandaRiu
Copy link
Contributor

@AmandaRiu AmandaRiu commented Jun 5, 2021

Fixes #3316

Gutenberg PR: WordPress/gutenberg#32471

To test:
See testing instructions in the gutenberg PR.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 5, 2021

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@AmandaRiu AmandaRiu requested a review from mkevins June 16, 2021 18:57
@AmandaRiu AmandaRiu removed the request for review from mkevins June 16, 2021 19:03
RELEASE-NOTES.txt Outdated Show resolved Hide resolved
@AmandaRiu AmandaRiu requested a review from dcalhoun June 16, 2021 21:34
Copy link
Contributor

@mkevins mkevins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested through the gutenberg PR. LGTM!

@AmandaRiu AmandaRiu removed the request for review from dcalhoun June 16, 2021 23:01
@AmandaRiu AmandaRiu dismissed dcalhoun’s stale review June 16, 2021 23:03

Another review not needed as approved by Matthew Kevins and these are small changes.

@AmandaRiu AmandaRiu merged commit 3c2dcdd into develop Jun 16, 2021
@AmandaRiu AmandaRiu deleted the rnmobile/fix-gboard-enter-detection branch June 16, 2021 23:03
@geriux geriux mentioned this pull request Jun 25, 2021
5 tasks
mokagio added a commit to wordpress-mobile/WordPress-iOS that referenced this pull request Jun 29, 2021
See title in the PR that was linked to it
(wordpress-mobile/gutenberg-mobile#3590):

> [Android] Fix Gboard enter detection in EnterPressedWatcher

If you then follow up on the Gutenberg PR
(WordPress/gutenberg#32471)
that actually made the change, you'll see confirmation it's Android
only:

> [RNMobile] Android Only - Fix Gboard enter detection in
> EnterPressedWatcher
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.

Enter detection fails in EnterPressedWatcher for Gboard keyboard
3 participants