-
Notifications
You must be signed in to change notification settings - Fork 82
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
Allow propagation #82
Comments
we need it there, try to handle bootstrap click above ui-cropper |
if you can't handle, you can add option to disable it, so by default it should be enabled. PR are welcome, just update also docs |
Optional. A boolean determining whether to allow 'mousedown' event propagation. CrackerakiUA#82
@CrackerakiUA I've created PR, but I didn't get where to update the docs. I've added description to PR. |
@CrackerakiUA actually it's not clear for me how to create PR on the wiki. I've create a new repo and made changes here: https://github.com/ViieeS/ui-cropper-wiki. Also, please make |
I try to use uiCropper and ui.bootstrap.contextMenu together.
The problem is after menu opened it doesn't close with left or right click because of this line:
ui-cropper/compile/unminified/ui-cropper.js
Line 2507 in 9c26040
@CrackerakiUA, can you please explain why we use it here? Maybe we can allow event propagation optionally?
The text was updated successfully, but these errors were encountered: