-
Notifications
You must be signed in to change notification settings - Fork 7
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
fix: replace selection polyfill in quill to handle Safari selection issues #179
Conversation
c34b486
to
1f8abd7
Compare
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.
I checked RTE demo pages on various versions of iOS (10.3, 11.2, 12.1,13.3, 14.3) and it worked as expected and no problem was seen. Also other browsers (IE11, FF, Edge) were tested and it was working fine, both RTE functionality and also adding an extra input field on the page to test typing into input field.
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.
LGTM 🎉
Description
Updates Quill dependency to use a new polyfill for shadow DOM text selection (related PR: vaadin/quill#2).
Fixes: vaadin/web-components#2115
Fixes: vaadin/web-components#2139 - this one is about text input and selection not working in any Vaadin component in iOS 12, which is also caused by the old polyfill