set data in pushState to load page with tab param

dradis · Jun 24, 2024 · 489b982 · 489b982
1 parent 28fee46
commit 489b982
Showing 1 changed file with 5 additions and 11 deletions.
diff --git a/app/assets/javascripts/shared/behaviors.js b/app/assets/javascripts/shared/behaviors.js
@@ -87,17 +87,11 @@
         let currentTab = $(e.target).attr('href').substring(1);
         searchParams.set('tab', currentTab);
         let urlWithTab = `?${searchParams.toString()}`;
-        history.pushState(null, null, urlWithTab);
-      });
-
-    // Allows users to navigate using the native browser back/forward buttons
-    // even when we manipulate the browser history with pushState()
-    $(window)
-      .off()
-      .on('popstate', function () {
-        if (location.search.length) {
-          Turbolinks.visit(location, { action: 'replace' });
-        }
+        history.pushState(
+          { turbolinks: true, url: urlWithTab },
+          '',
+          urlWithTab
+        );
       });
   }