Use shortened URL in pushState

Earlier we were using the full URL, now only query string is passed in pushState, much cleaner!
kiwix · Jun 24, 2022 · dcd2a49 · dcd2a49
1 parent 7f409e3
commit dcd2a49
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/static/skin/index.js b/static/skin/index.js
@@ -357,7 +357,7 @@
         params = new URLSearchParams(window.location.search);
         if (filterType) {
             params.set(filterType, filterValue);
-            window.history.pushState({}, null, `${window.location.href.split('?')[0]}?${params.toString()}`);
+            window.history.pushState({}, null, `?${params.toString()}`);
             setCookie(filterCookieName, params.toString());
         }
         updateFilterColors();
@@ -468,8 +468,8 @@
         const tagElement = document.getElementsByClassName('tagFilterLabel')[0];
         tagElement.addEventListener('click', () => removeTagElement(true));
         if (filters) {
-            const currentLink = window.location.href;
-            const newLink = `${window.location.href.split('?')[0]}?${params.toString()}`;
+            const currentLink = window.location.search;
+            const newLink = `?${params.toString()}`;
             if (currentLink != newLink) {
                 window.history.pushState({}, null, newLink);       
             }