Modernizing (lol) to use popstate event; this.history[] will keep…

… track of the nav history, updating README.md
avoidwork · Apr 16, 2021 · c19552e · c19552e
1 parent c79573a
commit c19552e
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -94,10 +94,10 @@ Multi-tier routing delimiter, defaults to `/`, e.g. `#settings/billing`; each ti
 
 ## API
 #### current()
-Returns the current `Route`, if logging is enabled
+Returns the current `Route`; if logging is enabled the trigger `Element` will be present
 
-#### popstate(ev)
-Event handler, expects `{oldURL: "", newURL: ""}`
+#### popstate()
+Event handler
 
 #### scan(default)
 Scans `ctx` for routes & resets `default` which is an optional argument, otherwise it defaults to the first route
@@ -108,6 +108,7 @@ Context specific DOM selector
 
 ## Requirements
 - `Element.classList` API, or shim
+- `popstate` Event
 
 ## License
 Copyright (c) 2021 Jason Mulligan

diff --git a/lib/dom-router.js b/lib/dom-router.js
@@ -135,7 +135,7 @@
 
 			if (!has(this.css.hidden, this.ctx.classList)) {
 				if (hash.length > 0 && includes(this.routes, hash)) {
-					this.handler({oldURL: "", newURL: document.location.hash});
+					this.handler();
 				} else {
 					this.route(this.start);
 				}

diff --git a/lib/dom-router.min.js b/lib/dom-router.min.js
diff --git a/lib/dom-router.min.js.map b/lib/dom-router.min.js.map
diff --git a/src/router.js b/src/router.js
@@ -107,7 +107,7 @@
 
 			if (!has(this.css.hidden, this.ctx.classList)) {
 				if (hash.length > 0 && includes(this.routes, hash)) {
-					this.handler({oldURL: "", newURL: document.location.hash});
+					this.handler();
 				} else {
 					this.route(this.start);
 				}

diff --git a/test/www/dom-router.js b/test/www/dom-router.js
@@ -135,7 +135,7 @@
 
 			if (!has(this.css.hidden, this.ctx.classList)) {
 				if (hash.length > 0 && includes(this.routes, hash)) {
-					this.handler({oldURL: "", newURL: document.location.hash});
+					this.handler();
 				} else {
 					this.route(this.start);
 				}