-
Notifications
You must be signed in to change notification settings - Fork 358
fix(ios): fix keyboard displacement bug in iOS 12 WKWebView #75
Conversation
any hope to see this included so I don't have to hack the plugin too? Moving to ionic is not a possibility without breaking local store so we need this fix for building in xcode 10. thx |
Curious about this too. Just start experiencing the keyboard issue as well. |
Why It's not fixed yet? Can any one provide me solution for this issue? |
CI, please rerun tests. |
And once more. |
Can we look forward to this soon, given that all tests are passing? |
Not from me, I don't review and approve (and merge) PRs that I don't fully understand (as is the case for all iOS code). Some other contributor will have to do that, I just made sure the tests were run successfully. (You can of course just use https://github.com/mwchambers/cordova-plugin-wkwebview-engine from github directly until then.) |
I don’t think this is a valid fix. If the page is scrolled, like when the entire form doesn’t fit in the view, the page will scroll to the top each time you interact with an input-element. This will quickly become extremely tiresome… ;) This means: Click a dropdown, select a value, [ ScrolledToTop ], scrolling down, select another field, close the keyboard, [ ScrolledToTop ], rinse-and-repeat. |
I had not noticed this, however you are absolutely correct! I've just tested it and it does behave how you've described. I guess https://github.com/ionic-team/cordova-plugin-ionic-webview/ must have the same issue. Unfortunately I don't have the experience required to help with this any further. |
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.
Do not merge: #75 (comment)
This is resolved in iOS 13.4. For earlier versions of iOS, using the cordova-plugin-wkkeyboardfix is probably the best solution. We will not be making any changes to Cordova to address this bug. |
Using solution from here:
https://github.com/ionic-team/cordova-plugin-ionic-webview/pull/201/commits
fix #71