From 489b982d4387894490c86ef25d9d13c06d5151d7 Mon Sep 17 00:00:00 2001
From: Matt Budz <matt.budz@protonmail.com>
Date: Mon, 24 Jun 2024 18:11:12 +0200
Subject: [PATCH] set data in pushState to load page with tab param

---
 app/assets/javascripts/shared/behaviors.js | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/app/assets/javascripts/shared/behaviors.js b/app/assets/javascripts/shared/behaviors.js
index 29bdaa231..9aa1e52a0 100644
--- 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
+        );
       });
   }