Stop the history panel buttons from submitting their forms

Both buttons sit in a form and carry no type, so they default to submit.
They only avoid navigating because history.js calls preventDefault, and
that script loads with the panel, so a click landing first sends the
browser to the raw JSON of the sidebar or refresh view. The toolbar is
then gone from the page, which is what the switch test hit on CI.

The forms are read by ajaxForm rather than submitted, so nothing needs
the submit behaviour. Clicking early is now a no-op instead, so the test
retries the click until the switch takes effect.

Author: novucs

debug_toolbar/templates/debug_toolbar/panels/history.html

@@ -1,7 +1,7 @@
 {% load i18n static %}
 <form method="get" action="{% url 'djdt:history_refresh' %}">
     {{ refresh_form.as_div }}
-    <button class="refreshHistory">Refresh</button>
+    <button type="button" class="refreshHistory">Refresh</button>
 </form>
 <table class="djdt-max-height-100">
     <thead>

debug_toolbar/templates/debug_toolbar/panels/history_tr.html

@@ -44,7 +44,7 @@
     <td class="djdt-actions">
         <form method="get" action="{% url 'djdt:history_sidebar' %}">
             {{ history_context.form.as_div }}
-            <button data-request-id="{{ request_id }}" class="switchHistory">Switch</button>
+            <button type="button" data-request-id="{{ request_id }}" class="switchHistory">Switch</button>
         </form>
     </td>
 </tr>

docs/changes.rst

@@ -13,6 +13,8 @@ Pending
   parameters.
 * Fixed the error shown when panel content fails to load, which could not
   find the toolbar window inside the shadow root.
+* Stopped the history panel buttons from submitting their form when clicked
+  before the panel script has loaded, which navigated away from the page.
 
 7.0.0 (2026-06-17)
 ------------------

tests/test_integration.py

@@ -788,16 +788,30 @@ def test_rerender_on_history_switch(self):
         self.get("/regular_jinja/basic")
         # Make a new request so the history panel has more than one option.
         self.get("/execute_sql/")
-        template_panel = self.selenium.find_element(By.ID, HistoryPanel.panel_id)
         # Record the current side panel of buttons for later comparison.
+        previous_request_id = self.selenium.find_element(
+            By.ID, "djDebug"
+        ).get_attribute("data-request-id")
         previous_button_panel = self.selenium.find_element(
             By.ID, "djDebugPanelList"
         ).text
 
         # Click to show the history panel
         self.selenium.find_element(By.CLASS_NAME, HistoryPanel.panel_id).click()
+
         # Click to switch back to the jinja page view snapshot
-        list(template_panel.find_elements(By.CSS_SELECTOR, "button"))[-1].click()
+        def switch_to_oldest_snapshot(selenium):
+            buttons = selenium.find_element(By.ID, HistoryPanel.panel_id).find_elements(
+                By.CSS_SELECTOR, ".switchHistory"
+            )
+            if buttons:
+                buttons[-1].click()
+            return (
+                selenium.find_element(By.ID, "djDebug").get_attribute("data-request-id")
+                != previous_request_id
+            )
+
+        self.wait.until(switch_to_oldest_snapshot)
 
         current_button_panel = self.selenium.find_element(
             By.ID, "djDebugPanelList"