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

Revert "[iOS] Send connectionClosed message when resignFirstResponder to ensure framework focus state is correct." #40777

Merged
merged 1 commit into from Mar 30, 2023

Conversation

luckysmg
Copy link
Contributor

@luckysmg luckysmg commented Mar 30, 2023

Reverts #40703

I found that since the connectionClosed will invoke _finalizeEditing(TextInputAction.done, shouldUnfocus: true);.
This will invoke onSubmit.....But actually we do expect just unfocus and do not expect any submit.. This will cause some break change. Need to have further invesigate to reland

… to ensure framework focus state is correct. (#40703)"

This reverts commit bad50dd.
@luckysmg luckysmg added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 30, 2023
@auto-submit auto-submit bot merged commit 75fabbe into main Mar 30, 2023
37 checks passed
@auto-submit auto-submit bot deleted the revert-40703-handle-unfocus branch March 30, 2023 08:11
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 30, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Mar 30, 2023
…tResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (#123760)

Roll Flutter Engine from 14e536b8c29c to 75fabbecbe1f (1 revision)
sourcegraph-bot pushed a commit to sgtest/megarepo that referenced this pull request Mar 30, 2023
…esignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (#123760)

Commit: fe7b6715be6faf5e44b9ee7fdaa9f77ef1cc0753
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
luckysmg added a commit that referenced this pull request Apr 10, 2023
…esponder to ensure framework focus state is correct. (#40703)" (#40777)"

This reverts commit 75fabbe.
exaby73 pushed a commit to NevercodeHQ/flutter that referenced this pull request Apr 17, 2023
…tResponder to ensure framework focus state is correct. (flutter#40703)" (flutter/engine#40777) (flutter#123760)

Roll Flutter Engine from 14e536b8c29c to 75fabbecbe1f (1 revision)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
…resignFirstResponder to ensure framework focus state is correct. (#40703)" (flutter/engine#40777) (flutter/flutter#123760)
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 platform-ios
Projects
None yet
2 participants