From d478aad408de250396c860093dbe27b10c021d24 Mon Sep 17 00:00:00 2001
From: Krzysztof Kotlarek <kotlarek.krzysztof@gmail.com>
Date: Wed, 11 Jun 2025 12:19:17 +0800
Subject: [PATCH 1/2] FIX: edit button not working

After this PR <a> link stopped working correctly page started doing a full reload. <Button> component behaves correctly.

https://github.com/discourse/discourse-data-explorer/pull/376
---
 .../admin/plugins-explorer-queries-details.gjs       | 12 +++++-------
 assets/stylesheets/explorer.scss                     |  2 +-
 spec/system/explorer_spec.rb                         |  9 +++++++++
 3 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs b/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs
index 1e7c6835..1733c27d 100644
--- a/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs
+++ b/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs
@@ -61,13 +61,11 @@ export default RouteTemplate(
             <h1>
               {{@controller.model.name}}
               {{#unless @controller.editDisabled}}
-                <a
-                  href
-                  {{on "click" @controller.editName}}
-                  class="edit-query-name"
-                >
-                  {{icon "pencil"}}
-                </a>
+                <DButton
+                  @action={{@controller.editName}}
+                  @icon="pencil"
+                  class="edit-query-name btn-transparent"
+                />
               {{/unless}}
             </h1>
           </div>
diff --git a/assets/stylesheets/explorer.scss b/assets/stylesheets/explorer.scss
index de474824..7c07d873 100644
--- a/assets/stylesheets/explorer.scss
+++ b/assets/stylesheets/explorer.scss
@@ -215,7 +215,7 @@ table.group-reports {
       margin: 0 0.5em 0 0;
       color: var(--primary);
 
-      a {
+      button .d-icon {
         color: currentcolor;
       }
     }
diff --git a/spec/system/explorer_spec.rb b/spec/system/explorer_spec.rb
index bb99da21..bc8ae339 100644
--- a/spec/system/explorer_spec.rb
+++ b/spec/system/explorer_spec.rb
@@ -27,6 +27,15 @@
 
       expect(page).to have_field("limit", with: 42)
     end
+
+    it "allows to edit custom name" do
+      visit("/admin/plugins/explorer/queries/#{query_1.id}")
+      find(".query-run .btn-primary").click
+      find(".edit-query-name").click
+      find(".name-text-field input").fill_in(with: "My custom name edited")
+      find(".btn-primary").click
+      find("button span", text: "Save Changes and Run").click
+    end
   end
 
   context "with the old url format" do

From c1d1546f8880aad8fc06219980915fd462442ec0 Mon Sep 17 00:00:00 2001
From: Krzysztof Kotlarek <kotlarek.krzysztof@gmail.com>
Date: Wed, 11 Jun 2025 13:39:03 +0800
Subject: [PATCH 2/2] FIX: tests

---
 .../templates/admin/plugins-explorer-queries-details.gjs        | 2 +-
 spec/system/explorer_spec.rb                                    | 1 +
 test/javascripts/acceptance/run-query-test.js                   | 2 +-
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs b/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs
index 1733c27d..0d412d23 100644
--- a/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs
+++ b/assets/javascripts/discourse/templates/admin/plugins-explorer-queries-details.gjs
@@ -59,7 +59,7 @@ export default RouteTemplate(
             />
 
             <h1>
-              {{@controller.model.name}}
+              <span>{{@controller.model.name}}</span>
               {{#unless @controller.editDisabled}}
                 <DButton
                   @action={{@controller.editName}}
diff --git a/spec/system/explorer_spec.rb b/spec/system/explorer_spec.rb
index bc8ae339..4449eb55 100644
--- a/spec/system/explorer_spec.rb
+++ b/spec/system/explorer_spec.rb
@@ -35,6 +35,7 @@
       find(".name-text-field input").fill_in(with: "My custom name edited")
       find(".btn-primary").click
       find("button span", text: "Save Changes and Run").click
+      expect(page.find(".name h1")).to have_content("My custom name edited")
     end
   end
 
diff --git a/test/javascripts/acceptance/run-query-test.js b/test/javascripts/acceptance/run-query-test.js
index d9095bf2..5b8ce6f1 100644
--- a/test/javascripts/acceptance/run-query-test.js
+++ b/test/javascripts/acceptance/run-query-test.js
@@ -320,7 +320,7 @@ acceptance("Data Explorer Plugin | Run Query", function (needs) {
     await visit("/admin/plugins/explorer/queries/2");
 
     assert
-      .dom("div.name h1")
+      .dom("div.name h1 span")
       .hasText("What about 0?", "the query name was rendered");
 
     assert