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

Conversation

@gspencergoog
Copy link
Contributor

@gspencergoog gspencergoog commented Jan 13, 2020

Description

This changes the InputConnectionAdaptor so that it will execute an IME action when ENTER is pressed. Prior to this, pressing ENTER on a hardware keyboard did nothing.

Related Issues

Tests

  • Added tests for method calls when keys are pressed, and fixed a test that thought it was checking the method call, but wasn't.

Copy link
Member

@gaaclarke gaaclarke left a comment

Choose a reason for hiding this comment

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

lgtm

@gspencergoog gspencergoog force-pushed the fix_keyboard_enter branch 2 times, most recently from 2ccaf38 to 6e8aaf3 Compare January 14, 2020 01:36
@gspencergoog gspencergoog force-pushed the fix_keyboard_enter branch 2 times, most recently from 15a923a to 4e8f63e Compare January 14, 2020 02:21
@gspencergoog gspencergoog requested review from mklim and removed request for GaryQian January 14, 2020 21:51
@gspencergoog gspencergoog force-pushed the fix_keyboard_enter branch 2 times, most recently from d256a63 to 2a2c79b Compare January 14, 2020 21:55
Copy link
Contributor

@mklim mklim left a comment

Choose a reason for hiding this comment

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

LGTM

@gspencergoog gspencergoog merged commit 73cdd0e into flutter:master Jan 15, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 16, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 16, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 16, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 16, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 16, 2020
flutter/engine@3488051...be20fb6

git log 3488051..be20fb6 --first-parent --oneline
2020-01-15 [email protected] [web] Exec command copy (flutter/engine#15675)
2020-01-15 [email protected] Disable fml_unittests till flakes are addressed. (flutter/engine#15676)
2020-01-15 [email protected] Roll src/third_party/dart fe666ce592cb..862d5012ae9c (48 commits) (flutter/engine#15672)
2020-01-15 [email protected] Fix hardware keyboard enter so it triggers an action. (flutter/engine#15568)
2020-01-15 [email protected] Use iOS 13 dark content status bar style (flutter/engine#13119)
2020-01-15 [email protected] Roll fuchsia/sdk/core/linux-amd64 from oxHfW... to UlOSN... (flutter/engine#15670)
2020-01-15 [email protected] Roll src/third_party/skia e45c5cd03eeb..7655168e6865 (29 commits) (flutter/engine#15671)
2020-01-15 [email protected] Fix embedder mutation order (flutter/engine#15566)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
This changes the InputConnectionAdaptor so that it will execute an IME action when ENTER is pressed. Prior to this, pressing ENTER on a hardware keyboard did nothing.
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
filmil pushed a commit to filmil/engine that referenced this pull request Mar 13, 2020
This changes the InputConnectionAdaptor so that it will execute an IME action when ENTER is pressed. Prior to this, pressing ENTER on a hardware keyboard did nothing.
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.

onSubmitted with TextField when pressing Enter key

4 participants