Skip to content
This repository has been archived by the owner on Mar 10, 2024. It is now read-only.

Enable dnd-nodrag on the same element as dnd-draggable #358

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

Enable dnd-nodrag on the same element as dnd-draggable #358

wants to merge 1 commit into from

Conversation

bogdanalexe90
Copy link

When you use both directives (dnd-draggable and dnd-nodrag) on the same element, dnd-nodrag will not work as expected because it will only prevent the dragstart and dragend events from bubbling up but won't prevent the rest of element event handlers from firing up.

When you use both directives (dnd-draggable and dnd-nodrag) on the same element, dnd-nodrag will not work as expected because it will only prevent the `dragstart` and `dragend` events from bubbling up but won't prevent the rest of element event handlers from firing up.
@marceljuenemann
Copy link
Owner

Thanks for your PR! This is an elegant solution I did not know about 👍

@marceljuenemann marceljuenemann added this to the 2.1.0 milestone Dec 25, 2016
@marceljuenemann marceljuenemann changed the title Fixed unwanted execution of dragStart event handler inside dndDraggable directive Enable dnd-nodrag on the same element as dnd-draggable Dec 26, 2016
@marceljuenemann marceljuenemann modified the milestones: 2.1.0, 2.x Dec 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants