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

fix screen freeze bug when keyboard is shown and textfield doesn't move #1105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HajimeNakahara
Copy link

this is the pr for fixing screen freeze when keyboard is shown and textfield doesn't move.

@pichillilorenzo
Copy link
Owner

@HajimeNakahara sorry, but I didn't understand when it happens. Could you provide a more detailed explanation and an example? Thanks

@biner88
Copy link

biner88 commented Feb 8, 2023

Very useful, please merge @pichillilorenzo

@kidwar
Copy link

kidwar commented May 31, 2023

@pichillilorenzo
Step to reproduce:

-> Click on a textfield inside webview‬ -> keyboard will be show up -> Click on another textfield -> Try to scroll bottom of page -> Can't scroll to bottom of page because total height of page is wrong now.

Very useful, please merge!

@pichillilorenzo
Copy link
Owner

@kidwar I am still not able to replicate the issue.
If I use https://github.com/login as initial URL example of an InAppWebView, I'm able to switch between username and password text inputs and still scroll the page without any issue.
If I test the code of this pull request, when the keyboard shows up, it adds unnecessary scrollable empty space to the end of the webpage.
Please, if you can, post a complete code example that can replicate the issue, otherwise I cannot test and find the right solution for the problem, if it is a real problem of the WebView first of all.

@kidwar
Copy link

kidwar commented Jun 13, 2023

@pichillilorenzo can you test with https://websitesandapps.co.uk/form.html as initial URL example

Video demo:
flutter_inappwebview: 6.0.0-beta.24

RPReplay_Final1686645673.MP4

@RMatushkin
Copy link

Any updates here?

@prateekrk
Copy link

something similar to the issue i have raised 2049?

@arrrrny

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants