This repository was archived by the owner on Feb 25, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 6k
Calling onConnectionClosed when the input element is blurred #14484
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mdebbar
reviewed
Dec 13, 2019
Contributor
mdebbar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You probably want to close the other PR: #13344 :)
yjbanov
approved these changes
Dec 13, 2019
Contributor
Author
Thanks I closed it now :) |
Contributor
Author
|
I changed the timer with document.hasFocus and document.activeElement. Please have another look. |
yjbanov
approved these changes
Dec 17, 2019
mdebbar
approved these changes
Dec 17, 2019
Contributor
mdebbar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
This was referenced Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
This was referenced Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Dec 18, 2019
chingjun
pushed a commit
to flutter/flutter
that referenced
this pull request
Dec 18, 2019
* ee92285 Roll src/third_party/skia fdb2b7d53038..cbeabd969bbe (1 commits) (flutter/engine#14530) * 9b369ad Rename 'test_vsync_waiters' to 'vsync_waiters_test' for consistency. (flutter/engine#14519) * 0f97a1e Roll src/third_party/skia cbeabd969bbe..3794db84e141 (8 commits) (flutter/engine#14532) * ac42640 Roll src/third_party/dart f10fa3df28a8..44da51fc720d (20 commits) (flutter/engine#14533) * c216ba4 Calling onConnectionClosed when the input element is blurred (flutter/engine#14484) * 44f0757 Manual roll of Dart to revision 8a4cac1ddb141d4a1e395f6e0a7d5bbf2247eb63 (flutter/engine#14542) * 995c4a7 Roll src/third_party/dart 8a4cac1ddb14..b6e051e2c87a (2 commits) (flutter/engine#14545) * c568308 Roll src/third_party/skia 190b82d67ce1..37a88b43637e (1 commits) (flutter/engine#14546) * c327cae Revert "Fix off-by-one fromRGBO alpha value calculation (#13777)" (flutter/engine#14548)
filmil
pushed a commit
to filmil/engine
that referenced
this pull request
Mar 13, 2020
…#14484) * close connection call * closing connection on blur * remove the timer and check the window focus directly. address reviewer comments. * addressing reviewer comments
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Showing the usage of framework PR: flutter/flutter#43466
This is the same implementation in #13344 Sent as a new PR since the class went through a major refactor.
Fixes flutter/flutter#43034
Tested manually for:
(**): known issue since there is only one persistent mode now. The mobile browsers are also showing desktop optimized behavior. depending on the decision, will send a follow up PR which will also fix the known issue for wrong transform for input elements.