Stop propagation of click events for onClickClose and the close button

[Matty666]

I added a stopPropagation function to the noty object, and called it from the $bar click event and the $closeButton click event to stop the event propagating.

[KasimAhmic]

Is this code supposed to solve this:
http://kasimahmic.koding.com/works/Notes/
Click "Delete" on any of the items and the "Ok". Click "Delete" on another item and the amount of noty dialogs doubles each time.

If so, how exactly do I enable it? I'm using your code but I'm still getting this problem :/

[Matty666]

@TheInfection the changes I made only address the issue I was having with the 'closeWith' settings. Once you change the 'buttons' setting to an array of buttons, this overrides the 'closeWith' settings. The issue I was having was when I set 'closeWith: ["button","click"]' and 'buttons: false', clicking the default close button would also click through and trigger the onCloseClick.

The issue you have is with your code, every time an ajax call completes, you are adding a new click handler to the delete button, so when you click it, it fires each handler you have added, which then does an ajax call, which completes, which then adds a new click handler.

[KasimAhmic]

@DJMatty Damn. Could you help me fix my problem? I have no idea what to do nor where to start :/

[Matty666]

Use a debugger and step though the code. When any ajax call completes you call .on for each ".delete" element, and .on for each ".note container p" element. This means that each of these ajax calls adds that function again to each element.

Perhaps you could look at trying to check if the function is already bound to that event? Or call .off first before you call .on to remove the old event handler and add the new event handler. I don't think this is the right place for us to be having a conversation about your code that really has nothing to do with noty, perhaps we should continue this conversation elsewhere?

[KasimAhmic]

@DJMatty Alright, I figured it out. I ended up using event.stopPropagation() and it works perfectly now. Thanks for pointing me in the right direction!