[BUILD-868] Fix button state when closing chatbot using X (#1042)

AnkiHubSoftware · Nov 29, 2024 · 1c786fe · 1c786fe
1 parent 059c811
commit 1c786fe
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/ankihub/gui/web/ankihub_ai.js b/ankihub/gui/web/ankihub_ai.js
@@ -94,6 +94,7 @@ class AnkiHubAI {
     hideIframe() {
         this.iframe.style.display = "none";
         this.iframeVisible = false;
+        window.ankihubReviewerButtons.unselectAllButtons(false);
     }
 
     cardChanged(noteId) {

diff --git a/ankihub/gui/web/reviewer_buttons.js b/ankihub/gui/web/reviewer_buttons.js
@@ -92,7 +92,7 @@ class AnkiHubReviewerButtons {
         buttonContainer.style.flexDirection = "column";
     }
 
-    setButtonState(buttonData, buttonElement, active) {
+    setButtonState(buttonData, buttonElement, active, sendToPython = true) {
         buttonData.active = active;
         if (active) {
             buttonElement.style.backgroundColor = this.theme == "light" ? this.colorButtonSelectedLight : this.colorButtonSelectedDark;
@@ -101,16 +101,17 @@ class AnkiHubReviewerButtons {
             buttonElement.style.backgroundColor = this.theme == "light" ? this.colorButtonLight : this.colorButtonDark;
         }
 
-
-        const args = `{"buttonName": "${buttonData.name}", "isActive": "${buttonData.active}"}`
-        pycmd(`ankihub_reviewer_button_toggled ${args}`);
+        if(sendToPython) {
+            const args = `{"buttonName": "${buttonData.name}", "isActive": "${buttonData.active}"}`;
+            pycmd(`ankihub_reviewer_button_toggled ${args}`);
+        }
     }
 
-    unselectAllButtons() {
+    unselectAllButtons(sendToPython = true) {
         for (const buttonData of this.buttonsData) {
             if (buttonData.active) {
                 const buttonElement = this.getButtonElement(buttonData.name);
-                this.setButtonState(buttonData, buttonElement, false);
+                this.setButtonState(buttonData, buttonElement, false, sendToPython);
             }
         }
     }