Merge pull request #823 from Johann-S/fixModalEvent

Create new event to avoid warning in Jasmine

1 files changed, 11 insertions, 10 deletions

diff --git a/js/modals.js b/js/modals.js
index db4f5bc..0012ca1 100644
--- a/js/modals.js
+++ b/js/modals.js
@@ -9,14 +9,6 @@
 !(function () {
   'use strict';
 
-  var eventModalOpen = new CustomEvent('modalOpen', {
-    bubbles: true,
-    cancelable: true
-  });
-  var eventModalClose = new CustomEvent('modalClose', {
-    bubbles: true,
-    cancelable: true
-  });
   var findModals = function (target) {
     var i;
     var modals = document.querySelectorAll('a');
@@ -40,9 +32,18 @@
   window.addEventListener('touchend', function (event) {
     var modal = getModal(event);
     if (modal && modal.classList.contains('modal')) {
-      var eventToDispatch = eventModalOpen;
+      var eventToDispatch = null;
       if (modal.classList.contains('active')) {
-        eventToDispatch = eventModalClose;
+        eventToDispatch = new CustomEvent('modalClose', {
+          bubbles: true,
+          cancelable: true
+        });
+      }
+      else {
+        eventToDispatch = new CustomEvent('modalOpen', {
+          bubbles: true,
+          cancelable: true
+        });
       }
       modal.dispatchEvent(eventToDispatch);
       modal.classList.toggle('active');