-
Notifications
You must be signed in to change notification settings - Fork 117
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
Safari Scrolls to Top on Focus #238
Comments
Thanks for reporting. The editor has a hidden input field which is used to capture quick keys like arrows to navigate through the contents of the editor. When you click somewhere inside the editor, this input field is given focus. This input field has a class name Anyone able to debug why Safari scrolls to top when giving this element focus and find a way to prevent Safari from scrolling up? (Chrome and Firefox do not do that). |
hi there ! just bumping this as it has been few months with no movement - this causes us daily issues and we haven't found any work around :( |
I think I have fixed this via 20129f8. Apparently Safari does do something special to the hidden input field when it has a width and height of zero 🤔 . |
Thank you so much ! I can't wait to test 😁 we really appreciate all of your hard work on this awesome package |
Fixed now in
Thanks! You're welcome. If the project is so important to you, please consider sponsoring me to ensure I can keep maintaining and improving the library 😄 |
When clicking on the JSON Editor near the end of a long JSON document, the browser force scrolls back to the top of the input requiring scrolling back down. I've managed to reproduce this in the Codesandbox for the React and Vue implementations:
To Reproduce:
I have created a fork of the linked React implementation just to add in a long example JSON, but you can reproduce this in the original linked React implementation as well if you'd like (you just need to add in enough JSON to need to scroll).
Additional Details
Thank you so much ! We love this library and are hoping for a fix for this as we use it for JSON inputs all over our application.
The text was updated successfully, but these errors were encountered: