MontoyaTech/ignite-html-router
MontoyaTech/ignite-html-router
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Removed the code that tries to reset the scroll position of the route.

Browse Source
This commit is contained in:
MattMo 2024-01-15 08:41:02 -08:00
parent ee1b5c5ca9
commit 28e85092c9
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
ignite-html-router.js
View File

@ -46,9 +46,6 @@ IgniteTemplate.prototype.route = function (routes, showCallback = null, hideCall
//Invoke the callback if the route matched.
if (routeMatches) {
//Replace the element so that the scroll position resets, if possible.
this.element.parentElement.parentElement.replaceChild(this.element.parentElement, this.element.parentElement);
//If the route is already visible, call hidden, because we are reshowing the route.
if (hideCallback && this.element.style.display != "none") {
hideCallback();
@ -345,7 +342,7 @@ class Router {
//Push the route to our internal states
Router.states.push(route);
//Create a new pushstate event and fire it.
//Fire a pushstate event to make the routes switch.
var event = new Event("pushstate");
event.data = data;
window.dispatchEvent(event);
@ -359,8 +356,11 @@ class Router {
*/
static back(fallback = null, refresh = false) {
if (Router.states && Router.states.length > 1) {
Router.states.pop(); //Pop the current state.
Router.navigate(Router.states.pop(), null, refresh); //Navigate to the previous state.
//Pop the current state.
Router.states.pop();
//Navigate to the previous state
Router.navigate(Router.states.pop(), null, refresh, false);
} else {
//Pop the current state, otherwise it can be used to go back but we don't want that.
if (Router.states.length > 0) {