Skip to content

Conversation

@LongCatIsLooong
Copy link
Contributor

@LongCatIsLooong LongCatIsLooong commented Sep 3, 2024

Fixes #154156

Some iOS keyboard implementations change the selection in the text field if dismissed with active composing regions. The framework should not call requestKeyboard in such cases since that would bring up the keyboard again.
In general the TextInput.show call is not needed for selection only changes. For working around #68571 the show call is needed only if we restarted the input on Android (and we don't restart on selection-only changes any way).

Pre-launch Checklist

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

@github-actions github-actions bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. labels Sep 3, 2024
Copy link
Contributor

@Renzo-Olivares Renzo-Olivares left a comment

Choose a reason for hiding this comment

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

LGTM

@LongCatIsLooong LongCatIsLooong added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 12, 2024
@auto-submit auto-submit bot merged commit dbb588b into flutter:master Sep 12, 2024
@LongCatIsLooong LongCatIsLooong deleted the unnecessary-showKeyboard branch September 12, 2024 21:30
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Sep 13, 2024
flutter/flutter@303f222...2d30fe4

2024-09-13 [email protected] Roll Packages from 91caa7a to 330581f (4 revisions) (flutter/flutter#155171)
2024-09-13 [email protected] Fix: Flicker when reorderable list doesn't change its position (flutter/flutter#151026)
2024-09-13 [email protected] Stop reading .packages from flutter_tools. (flutter/flutter#154912)
2024-09-13 [email protected] Roll Flutter Engine from 70109e3b40c0 to bef48e87f438 (1 revision) (flutter/flutter#155156)
2024-09-13 [email protected] Roll Flutter Engine from d917a15823f3 to 70109e3b40c0 (1 revision) (flutter/flutter#155151)
2024-09-13 [email protected] Roll Flutter Engine from 94696ed75dea to d917a15823f3 (1 revision) (flutter/flutter#155147)
2024-09-13 [email protected] Fix TextField content should be selected on desktop when gaining focus (flutter/flutter#154916)
2024-09-13 [email protected] Roll Flutter Engine from 04802b779045 to 94696ed75dea (1 revision) (flutter/flutter#155144)
2024-09-13 [email protected] Roll Flutter Engine from 3d8163f47919 to 04802b779045 (2 revisions) (flutter/flutter#155138)
2024-09-13 [email protected] Roll Flutter Engine from 4d5fea97e933 to 3d8163f47919 (1 revision) (flutter/flutter#155136)
2024-09-13 [email protected] Mark `_LayoutBuilderElement` as always clean (flutter/flutter#154694)
2024-09-13 [email protected] Roll Flutter Engine from 8609af642725 to 4d5fea97e933 (7 revisions) (flutter/flutter#155134)
2024-09-12 [email protected] Disable fuchsia in flutter_tools (flutter/flutter#155111)
2024-09-12 [email protected] Address frame policy benchmark flakes (flutter/flutter#155130)
2024-09-12 [email protected] Roll Flutter Engine from 48ddaf578fb0 to 8609af642725 (11 revisions) (flutter/flutter#155128)
2024-09-12 49699333+dependabot[bot]@users.noreply.github.com Bump peter-evans/create-pull-request from 7.0.1 to 7.0.2 (flutter/flutter#155126)
2024-09-12 [email protected] Prevent the keyboard from reshowing on iOS (flutter/flutter#154584)
2024-09-12 [email protected] fix(Linux): specify application id (flutter/flutter#154522)
2024-09-12 [email protected] update changelog on master (flutter/flutter#155109)
2024-09-12 [email protected] iOS: update provisioning profile for 2024-2025 cert (flutter/flutter#155101)
2024-09-12 [email protected] Roll Packages from 4c18648 to 91caa7a (2 revisions) (flutter/flutter#155103)

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] 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
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: text input Entering text in a text field or keyboard related problems autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[iPad] Clicking the dismiss button does not dismiss the keyboard when the composing region is active

2 participants