-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Make clickOutsideHandler use mousedown instead of mouseup #5391
Labels
Milestone
Comments
oleq
referenced
this issue
in ckeditor/ckeditor5-image
Jul 24, 2017
… of mouseup event (see ckeditor/ckeditor5-ui#281).
The event was changed to |
I don't remember. There was something related to clicking on toolbar buttons while LinkBalloon is opened but I'm not sure if this was a reason for this change. |
oskarwrobel
referenced
this issue
in ckeditor/ckeditor5-ui
Aug 7, 2017
Fix: The `clickOutsideHandler` helper should use `mousedown` instead of `mouseup` event. Closes #281.
oskarwrobel
referenced
this issue
in ckeditor/ckeditor5-image
Aug 7, 2017
Tests: The clickOutsideHandler helper should use mousedown instead of mouseup event (see ckeditor/ckeditor5-ui#281).
oskarwrobel
referenced
this issue
in ckeditor/ckeditor5-link
Aug 7, 2017
… of mouseup event (see ckeditor/ckeditor5-ui#281).
mlewand
added
status:confirmed
type:bug
This issue reports a buggy (incorrect) behavior.
package:ui
labels
Oct 9, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It looks like the wrong event makes a difference:
But why we chose
mouseup
it the first place? (cc @Reinmar)The text was updated successfully, but these errors were encountered: