Fix/notification message timeout (#201)

* Fix: notification context binding * Fix: don't use autobind * Fix: Cancel timeout on hide to avoid event stacking
box · Jul 7, 2017 · 5d42af4 · 5d42af4
1 parent a26d600
commit 5d42af4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/lib/Notification.js b/src/lib/Notification.js
@@ -56,7 +56,7 @@ class Notification {
         this.notificationEl.focus();
 
         // Hide notification automatically after a delay
-        setTimeout(this.hide, HIDE_TIMEOUT_MS);
+        this.timeout = setTimeout(this.hide.bind(this), HIDE_TIMEOUT_MS);
     }
 
     /**
@@ -65,6 +65,10 @@ class Notification {
      * @return {void}
      */
     hide() {
+        if (this.timeout) {
+            clearTimeout(this.timeout);
+            this.timeout = null;
+        }
         if (this.notificationEl) {
             this.notificationEl.classList.add(CLASS_HIDDEN);
         }