Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@dkwingsmt
Copy link
Contributor

Fixes flutter/flutter#108124.

The Android platform might dispatch a ShiftLeft key down event with SHIFT metaState 0. In order to resolve this conflict, this PR synthesizes a ShiftLeft up after the main event in such cases.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on
    writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@dkwingsmt dkwingsmt marked this pull request as ready for review July 21, 2022 23:31
@dkwingsmt dkwingsmt requested a review from gspencergoog July 22, 2022 00:06
Copy link
Contributor

@gspencergoog gspencergoog left a comment

Choose a reason for hiding this comment

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

32384589-a60f0e74-c078-11e7-9bc1-e5b5287aea9d

@dkwingsmt dkwingsmt merged commit a56d98e into flutter:main Jul 22, 2022
@dkwingsmt dkwingsmt deleted the android-key-post-synchronize branch July 22, 2022 01:16
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 22, 2022
@dkwingsmt dkwingsmt restored the android-key-post-synchronize branch August 30, 2022 00:43
@dkwingsmt dkwingsmt deleted the android-key-post-synchronize branch August 31, 2022 18:08
itsjustkevin pushed a commit that referenced this pull request Sep 20, 2022
…ting text in `TextField` (#36288)

* [Android] Synchronize incorrect metaState using post-synchronization (#34827)

(cherry picked from commit a56d98e)

* [Android] Fix a crash related to zeroed scanCode (#35924)

(cherry picked from commit fe5b84a)

Co-authored-by: Tong Mu <dkwingsmt@users.noreply.github.com>
Co-authored-by: Bruno Leroux <leroux_bruno@yahoo.fr>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Keyboard, Android] Modifier key_down can have 0 modifier, leading to crash

2 participants