-
Notifications
You must be signed in to change notification settings - Fork 25
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
dnd-draggable == "true" prevents ng-click #34
Comments
@Dimanoid hmm, in my case this working. Maybe i do something wrong? |
Stange but I cant reproduce it too on other computer. Btw I think that has an option to delay start dragging is good anyway... Count it as a feature request :) |
@Dimanoid hmm, i think yes, but need more info about this point :) can you describe it? |
Please check JQueryUI API, it has both options. |
I have the same problem. It begins drag when I click on element. And I can't check checkbox on child div. |
Yep its really annoying when you start dragging instead of just click because of cursor micro-movement. Can you please fix it anytime soon? Those new options will resolve this nicely. |
@hebedombiu, @Dimanoid You mean like 'tollerance movement' option? |
I think that I've found the problem on Chrome in Win10, this code seems to fix it: mousemove: function (event) {
if (event.movementX != 0 && event.movementY != 0) {
this.manipulator.progress(event);
}
} The problem is that Chrome fires |
@k-one-o-two good idea, but bad supporting for movementX property... |
Look at the example, if dnd-draggable set to true ng-click event never fired.
http://embed.plnkr.co/w5vP9BK3xVztYhVBe2OI/preview
You need to implement something like dnd-drag-delay option or dnd-drag-delta to start dragging after a few ms after onmousedown or after cursor were moved a few pixels away from start point with button hold pressed. Something similar is done in angular-ui-tree https://github.com/angular-ui-tree/angular-ui-tree
The text was updated successfully, but these errors were encountered: