-
Notifications
You must be signed in to change notification settings - Fork 17
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
unexpected scroll down when using navigation buttons on iOS devices #250
Comments
Hi,
... thus is not visible, and since it is rendered out of the normal document flow (thanks to I've no iOS device, so I cannot test by myself unfortunately. But what I would test first is to make the By the way, I'm opened to any solution allowing to capture the keyboard events without the need for this ugly hidden field. ;-) |
PS: you may have already notice it, but the problem arise in lib, and the problem may be easier to analyze by following the code in the kokopu-react repository (see file |
OK - that makes things a bit more clear. Maybe a stupid question, but why does the field need to be in focus?
And for obvious reasons, this does solve the page jump in iOS |
setting the position style to 'absolute' solves the issue, without commenting out any focus commands.
Alternatively, specifying position:absolute in a custom stylesheet also resolves this issue:
I will go for this last option as a quick fix already. Does it seem OK for you to modify the code to solve this issue natively? |
I'm testing the solution with Anyway, thank you for your search for a solution. I'll patch the kokopu-react lib accordingly (or you can also propose a pull request in https://github.com/yo35/kokopu-react). Just for the record, removing
Without |
The new version of RPB Chessboard with your fix is available. Thanks again. |
Requirements:
When using the navigation buttons, the page scrolls down unexpected, leaving the chessboard out of view.
I'm aware of #129, but don't manage to link the resolution there to the existing codebase.
Testing environment:
I'm trying to find the root cause, but in the meantime:
Reporting further when more insight gathered :).
The text was updated successfully, but these errors were encountered: