Bug: Multiple notifications, throws failed to execute 'removeChild'

ja1984 / jackbox

Javascript library to display notifications

https://ja1984.github.io/jackbox/

GNU General Public License v3.0

17 stars 6 forks source link

Bug: Multiple notifications, throws failed to execute 'removeChild' #13

Closed Fredrik-Oberg closed 8 years ago

Fredrik-Oberg commented 8 years ago

To reproduce :

Open two notifications
Close one of them manually

trevor-atlas commented 8 years ago

Any more details on reproducing this? I was unable to do so on latest chrome.

Fredrik-Oberg commented 8 years ago

@trevoristall In latest Chrome,

open one 'success' and one 'warning'
close 'success' before it auto closes
wait for 'warning' to auto close

Fredrik-Oberg commented 8 years ago

After some more investigation it can be reproduced by just opening any and then close it.

When the event mouseleave triggers it creates a new timeout. Could be that the event mouseleaves is triggered when we close the notification and it is animated down. Thus creating a new setTimeout, but the DOM element is already removed when that timer has run out.