CODEWITHSUNDEEP
javascriptjquery
user3032973
user3032973

Reputation: 415

Javascript prevent event propagation through element

Working with google maps which has infowindows that popup. These infowindows have images which are click-able. As a result I need to be able to propagate events in the infowindow. However I am also able to click THROUGH the infowindow to other markers which causes the current infowindow to close and it to open a new infowindow.

I dont think this issue is specific to google maps. Is there a way to stop events from propagating through an element?

Thought the below code would help but it didnt. 

$(document).on('touchstart', '.infoWindow', function(e){
  if ($(e.currentTarget) == $(this)) e.stopPropagation();
 });

Upvotes: 0

Views: 209

Answers (3)

Ohgyun Ahn
Ohgyun Ahn

Reputation: 787

Setting a background color of the .infoWindow element might be a solution. If it should be transparent, you can use as follows:

background-color: rgba(255, 0, 0, 0);

Upvotes: 0

Zenorbi
Zenorbi

Reputation: 2644

You cannot stop the event propagation when you use event delegation. Event delegation relies on the bubbleing event, so by the time the delegated handler gets the event, the event already bubbled through every other element. You need to attach your event listener directly to the element.

$('.infoWindow').on('touchstart', function(e){
    e.stopPropagation();
});

Event delegation works by adding the event listener to the document, then every time the document receives that touchstart event, jQuery checks the source element and all of its parent if they match the given selector. If one matches, the handler gets invoked. This is why you don't need to add the listener every time you add a new element that would match the given selector.

Upvotes: 1

Bergi
Bergi

Reputation: 665361

if ($(e.currentTarget) == $(this))

This is never true. It creates two distinct jQuery instances, which - even if they contain the same element - are not the same object. You might have wanted to do

if (e.currentTarget == this)

but this is always true - by definition of the event dispatch algorithm, the this value of an event listener and the currentTarget of the event object always refer to the same thing. Hell, even jQuery does this.

So you should just write

$(document).on('touchstart', '.infoWindow', function(e){
    e.stopPropagation();
});

to prevent touchstart events from bubbling through your infoWindows.

Upvotes: 2

Related Questions