zefyr icon indicating copy to clipboard operation
zefyr copied to clipboard

ListView doesn't scroll to ZefyrField when the software keyboard activates

Open JacobWrenn opened this issue 4 years ago • 9 comments

I have a ListView containing a TextField and a ZefyrField.

When I click into the TextField, the ListView scrolls up to ensure it isn't covered by the keyboard.

However, when I click into the ZefyrField, the ListView does nothing and therefore the ZefyrField gets covered by the keyboard.

The ZefyrField should scroll the ListView to respond to the keyboard in the same way the TextField does.

JacobWrenn avatar Feb 15 '21 21:02 JacobWrenn

web or mobile?

singerdmx avatar Feb 15 '21 21:02 singerdmx

On mobile. Specifically, iOS is where I am testing this.

JacobWrenn avatar Feb 15 '21 21:02 JacobWrenn

Do you want to give https://github.com/singerdmx/flutter-quill a try to see if you still have the issue? I am unsure what the issue is

singerdmx avatar Feb 15 '21 21:02 singerdmx

Screen recordings showing the desired behaviour of TextField and the contrasting behaviour of ZefyrField.

https://user-images.githubusercontent.com/37597713/108057905-2aed8780-704b-11eb-99f7-66051289d194.mov

https://user-images.githubusercontent.com/37597713/108057934-35a81c80-704b-11eb-90bb-3d39eb7a0eeb.mov

JacobWrenn avatar Feb 16 '21 11:02 JacobWrenn

https://github.com/flutter/flutter/issues/10826

singerdmx avatar Feb 21 '21 17:02 singerdmx

https://pub.dev/packages/keyboard_avoider

singerdmx avatar Feb 21 '21 18:02 singerdmx

I think I found a solution to this

https://stackoverflow.com/questions/60235386/is-it-possible-to-force-smooth-implicit-scrolling-for-flutters-accessibility-fo

We can call showOnScreen using _lastTapDownPosition

@override
handleTapDown(TapDownDetails details) {
  _lastTapDownPosition = details.globalPosition;
}

singerdmx avatar Feb 21 '21 19:02 singerdmx

Hello , we have fixed this issue on our repo, you can check the change here : https://github.com/singerdmx/flutter-quill/commit/471c1cb7ee542977b777b0fc10cbaa5cfb35ddea. Thanks!

li3317 avatar Feb 22 '21 04:02 li3317

Hello , we have fixed this issue on our repo, you can check the change here : singerdmx/flutter-quill@471c1cb. Thanks!

Hey, are you making an attempt to get this fix merged into this repo as it's the official one and v1 is in development?

britannio avatar Feb 23 '21 09:02 britannio