Skip to content

Add platform check to FocusManager app lifecycle listener - #144718

Merged
auto-submit[bot] merged 13 commits into
flutter:masterfrom
nate-thegrate:mobile-keyboard-fix
Mar 12, 2024
Merged

auto-submit[bot] merged 13 commits into
flutter:masterfrom
nate-thegrate:mobile-keyboard-fix

Conversation

@nate-thegrate

Copy link
Copy Markdown
Contributor

This PR implements a temporary fix for the mobile device keyboard bug reported in this comment.

CC @gspencergoog

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. f: focus Focus traversal, gaining or losing focus labels Mar 6, 2024
@nate-thegrate
nate-thegrate marked this pull request as ready for review March 6, 2024 21:24
Comment thread packages/flutter/lib/src/widgets/focus_manager.dart Outdated
Comment thread packages/flutter/lib/src/widgets/focus_manager.dart Outdated
Comment thread packages/flutter/test/widgets/focus_manager_test.dart Outdated
@gspencergoog

Copy link
Copy Markdown
Contributor

@nate-thegrate Could you write a quick test for this that makes sure it doesn't do anything on Android? That's all that is missing. We'd like to get this in, as it's a bit urgent for our customers (because their customers can't type in necessary information).

@nate-thegrate

Copy link
Copy Markdown
Contributor Author

@gspencergoog test has been added!

@gspencergoog gspencergoog left a comment

Copy link
Copy Markdown
Contributor

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

@justinmc justinmc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread packages/flutter/lib/src/widgets/focus_manager.dart
@gspencergoog gspencergoog added autosubmit Merge PR when tree becomes green via auto submit App and removed autosubmit Merge PR when tree becomes green via auto submit App labels Mar 12, 2024
@auto-submit
auto-submit Bot merged commit 61812ca into flutter:master Mar 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 12, 2024
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Mar 12, 2024
flutter/flutter@1ca8873...61812ca

2024-03-12 [email protected] Add platform check to `FocusManager` app lifecycle listener (flutter/flutter#144718)

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

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@nate-thegrate
nate-thegrate deleted the mobile-keyboard-fix branch March 14, 2024 04:06
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 14, 2024
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
flutter/flutter@1ca8873...61812ca

2024-03-12 [email protected] Add platform check to `FocusManager` app lifecycle listener (flutter/flutter#144718)

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

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
flutter/flutter@1ca8873...61812ca

2024-03-12 [email protected] Add platform check to `FocusManager` app lifecycle listener (flutter/flutter#144718)

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

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App f: focus Focus traversal, gaining or losing focus framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants