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

iOS 11.0.3, input box autosize is overlapping the keypad #343

Closed
enteeee opened this issue Nov 9, 2017 · 14 comments
Closed

iOS 11.0.3, input box autosize is overlapping the keypad #343

enteeee opened this issue Nov 9, 2017 · 14 comments

Comments

@enteeee
Copy link

enteeee commented Nov 9, 2017

This issue is happening on iOS 11.0.3 version (all browsers). When you try to type something on the text input focus is going down and input box is pulled down.

image12

@jhubble
Copy link

jhubble commented Nov 17, 2017

I ran into that problem also. I have a pull request that comments out the code that leads to the problem.

@enteeee
Copy link
Author

enteeee commented Nov 18, 2017

Sounds good, Does it work if I use npm install again?

@jhubble
Copy link

jhubble commented Nov 18, 2017

It should after the pull request is approved.
Until then, I have just been referring to my fork directly in package.json:
"autosize":"https://htihub.com/jhubble/autosize"

@enteeee
Copy link
Author

enteeee commented Nov 18, 2017

@jackmoore can u please approve this request.

@jackmoore
Copy link
Owner

@jhubble @enteeee I don't have a position on this yet, because it requires more consideration. Merging this would re-open #298. I'm open to suggestions, but there might not be a way to resolve both issues.

@aeharding
Copy link

aeharding commented Nov 20, 2017

Just curious since this is a regression with iOS -- does anyone know if there is a Webkit bug issue # for this?

I stumbled across this recently, and it's affecting more and more devices as users are upgrading.

My temporary fix in my project is to only disable autoprefixer for ios (progressive enhancement).

@jackmoore
Copy link
Owner

My temporary fix in my project is to only disable autoprefixer for ios (progressive enhancement).

This is what I would recommend as well, the script isn't that useful on mobile and disabling the scroll restore causes problematic behavior in desktop Safari.

@MiikaL
Copy link

MiikaL commented Nov 28, 2017

This problem also appears to only happen when the control is on the bottom of the screen, so if anybody needs a temporary fix, you can just add some empty space to your page.

aduth added a commit to aduth/dones that referenced this issue Dec 12, 2017
Replaces autosize, which presently has scroll issues in iOS 11. See: jackmoore/autosize#343
@eddieconk
Copy link

Any consideration had on this issue? Seems to be a large usability issue in some cases.

@razh
Copy link

razh commented Dec 17, 2017

Here's the issue in the WebKit Bugzilla:

https://bugs.webkit.org/show_bug.cgi?id=176896

And the change that should fix this:

https://trac.webkit.org/changeset/225715/webkit

@gena-tokarev
Copy link

gena-tokarev commented Jan 12, 2018

Here may be the fix #352
I didn't understand why it is necessary to check the body element for overflow
And seems that doesn't reopen #298

@jackmoore
Copy link
Owner

@Gasheck Autosize collapses the textarea element and uses the scrollHeight to figure out what the textarea height should be set to. This collapsing can cause scrolling ancestor elements to change their scroll position. To mitigate this the ancestor elements scroll positions are recorded and restored after the textarea element is given the calculated height.

@arggh
Copy link

arggh commented Nov 4, 2018

FYI: This seems to be fixed in iOS 11.4 (or earlier).

@jackmoore
Copy link
Owner

@arggh Thanks for the followup. I'm glad it's resolved, the only way I can think of to work around the issue would have opened up other issues for desktop browsers.

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

No branches or pull requests

9 participants