-
Notifications
You must be signed in to change notification settings - Fork 5k
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 the expression editor in recent versions of Chrome and Firefox #4969
Conversation
Hawt. That sorta kinda makes sense. Why is Safari different? Or is it just behind? |
Did anyone test it? I tried to patch this on 0.23.1 without any success. |
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.
Works for me as well on Chrome 58, Firefox 53.0 and Safari 10.1.
Just tried cherry-picking onto v0.23.1 and confirmed it works there too:
|
I tried the branch and it works locally. |
It worked! Not sure why it did not work earlier. Could have been cache on the browser side. |
Resolves #4906
Apparently in recent versions of FF and Chrome
window.getSelection().getRangeAt(0).setStart(element, 0);
will actually mutate the live selection, which is cool, but not what the previous code expected.