Merge pull request #2 from coverhound/jv-modifies-logic-for-hiding-da…

…tepicker-on-back-button-press Modifies logic to hide datepicker when clicking browser back button
jcgertig · May 22, 2017 · 96589cf · 96589cf
2 parents e4834be + d67c20a
commit 96589cf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/input.js b/input.js
@@ -89,12 +89,6 @@ export default class Input {
     });
   }
 
-  // when used in a single-page app,
-  // hide datepicker when the browser's back button is pressed
-  this.element.addEventListener("hashchange", () => {
-    thePicker.hide();
-  })
-
   getLocaleText() {
     const locale = this.locale.toLowerCase();
 

diff --git a/picker.js b/picker.js
@@ -129,6 +129,12 @@ class Picker {
       document.addEventListener(`mousedown`, this.removeClickOut);
       document.addEventListener(`touchstart`, this.removeClickOut);
     }, 500);
+
+    // when used in a single-page app  or otherwise,
+    // hide datepicker when the browser's back button is pressed
+    window.onpopstate = () => {
+      this.hide();
+    }
   }
 
   // Position picker below element. Align to element's right edge.