-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
RichText: remove iOS scroll adjusting #11971
Conversation
4e1b54d
to
74d1a48
Compare
Wow. This is hardcore impressive refactorings you've been doing. I love the amount of red in this PR! And the experience seems great to me: I think it's important we get a few more people testing this as a sanity check, remember it has to be a physical device, or the Xcode simulator. But first impression looks seriously promising. 👏👏👏👏👏 One issue that is completely unrelated, but becomes apparent again when testing this: we probably should never invoke "isEditing" mode on mobile, and always show the toolbar and selection borders. Reason being that it's otherwise both difficult and unintuitive to figure out how to make text bold. We are a bit limited here in what we can do because we are not a native app, and we are at the mercy of Mobile Safari (see also this) — but it still seems a better experience. But again, that's unrelated to this PR. REALLY nice work! |
Notably I'm super duper thrilled to see that the jumping toolbar is gone from this branch 💥 |
I have the same result as you shared in the video on a physical device. :) |
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.
Code removal looks legit 👍 I haven't tested but I trust you two :)
Tested it also on a different device. Thanks! |
So glad to see this code go. :) |
Description
With our new smooth enter behaviour (#11287), we no longer have a focus delay when creating a new paragraph on enter. It seems that we can remove the iOS fixes that adjust the scroll position when a block receives focus and when enter/delete is pressed.
Removing the fix actually results in even nicer scrolling because it is handled natively.
Reverts #5769.
How has this been tested?
Test scroll behaviour on iOS when pressing enter and backspace.
Screenshots
Types of changes
Checklist: