Skip to content
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

Do not send click up in dom tree. #183

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sebhofmann
Copy link
Contributor

This should fix #152

@tsanko
Copy link

tsanko commented Aug 25, 2015

+1

@faelcorreia
Copy link

+1
That did the trick for me!

Here two plunker with result before and after the change made by paschty:

http://plnkr.co/edit/xhLgXW0hPxLucQiKt6Q1?p=preview
http://plnkr.co/edit/jbZ7DqzUsfchAG3k0qmY?p=preview

@fracz
Copy link

fracz commented Oct 6, 2015

Please note that it can be fixed in client side only in desired situations by stopping the mouse down event propagation.

Just add the ng-mousedown="$event.stopPropagation()" to child element.

http://plnkr.co/edit/onwCzYy4mtI8pz1mSoNV?p=preview

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Draggable with Draggable Children? (draggable tree)
4 participants