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

Conversation

@nturgut
Copy link
Contributor

@nturgut nturgut commented Dec 13, 2019

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:

  1. Chrome on Linux desktop
  2. Chrome on Linux desktop a11y on with chromevox
  3. Firefox on Linux desktop
  4. Chrome on Android
  5. Chrome on Android a11y on with talkback (**)
  6. Safari on IOS
  7. Safari on IOS a11y on with voiceover (**)
  8. Safari on MacOS

(**): 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.

@nturgut nturgut requested review from mdebbar and yjbanov December 13, 2019 22:22
@auto-assign auto-assign bot requested a review from chinmaygarde December 13, 2019 22:22
@nturgut nturgut removed the request for review from chinmaygarde December 13, 2019 22:23
Copy link
Contributor

@mdebbar mdebbar left a 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 :)

@nturgut
Copy link
Contributor Author

nturgut commented Dec 16, 2019

You probably want to close the other PR: #13344 :)

Thanks I closed it now :)

@nturgut
Copy link
Contributor Author

nturgut commented Dec 17, 2019

I changed the timer with document.hasFocus and document.activeElement. Please have another look.

@nturgut nturgut requested review from mdebbar and yjbanov December 17, 2019 17:08
Copy link
Contributor

@mdebbar mdebbar left a comment

Choose a reason for hiding this comment

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

LGTM

@nturgut nturgut merged commit c216ba4 into flutter:master 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.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[web][Safari][mobile] Virtual Keyboard shifts the page up after browser closes/reopens

4 participants