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

stopPropagation doesn't work (but works with direct Hammer usage) #18

Closed
emirotin opened this issue Jun 20, 2014 · 5 comments
Closed

stopPropagation doesn't work (but works with direct Hammer usage) #18

emirotin opened this issue Jun 20, 2014 · 5 comments

Comments

@emirotin
Copy link

Here's the sample Plunker: http://plnkr.co/edit/7j2M0dFsk3swuAWKYsS9?p=preview
Whichever square you click the same text is shown: "Wrapper clicked"

To prove there's no mistake in code: go to script.js and comment out line 9. Now "Inner clicked" properly shows.

Now the same Plunker but with raw Hammer + jQuery (same Hammer version, 1.0.5): http://plnkr.co/edit/lIAb8tN8RqtYBikkMI3J?p=preview

To prove stopPropagation actually works here: go to stript.js and comment out line 17. Now it behaves as the 1st one.

BTW updating Hammer would also make sense, 1.0.5 is more than 1 year old.

@arthuryeti
Copy link

I have the same issue.

When I click on something, it does click on the underlying elements.

For exemple a modal with a form behind.

Do you have any ideas @wzr1337 ?

@emirotin
Copy link
Author

emirotin commented Sep 2, 2014

In addition looks that events are capturing instead of bubbling — I have the event on parent before it fires on child.

@arthuryeti
Copy link

@emirotin what do you mean by that ?

@jonnydungeons
Copy link

Have you found a fix for this severe bug? I would say this is important to fix because most angular apps are used in mobile / tablet setting and the ui typically have mobile-ish elements which reveal nested DOM and events.

@wzr1337
Copy link
Owner

wzr1337 commented Nov 10, 2014

closing here, continuing under

#23

@wzr1337 wzr1337 closed this as completed Nov 10, 2014
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

No branches or pull requests

4 participants