From f439705e712c6595197dafaeedb7d8af2eeeb2a1 Mon Sep 17 00:00:00 2001
From: Sj-Si <sjw.jetty@gmail.com>
Date: Tue, 30 Apr 2024 15:12:51 -0400
Subject: [PATCH] fix bug with longpress deselect

---
 javascript/extraNetworks.js | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/javascript/extraNetworks.js b/javascript/extraNetworks.js
index a69614a19..931820d7f 100644
--- a/javascript/extraNetworks.js
+++ b/javascript/extraNetworks.js
@@ -1375,15 +1375,17 @@ function extraNetworksSetupEventDelegators() {
         }
         // If long pressed, we deselect.
         // Else we set as long pressed.
-        if (elem.classList.contains("short-pressed")) {
-            elem.classList.remove("short-pressed");
-            elem.classList.add("long-pressed");
-            elem.dataset.selected = "";
-            elem.dataset.recurse = "";
+        if (elem.classList.contains("long-pressed")) {
+            elem.classList.remove("long-pressed");
+            delete elem.dataset.recurse;
+            // Don't want to remove selected state if btn was previously short-pressed.
+            if (!elem.classList.contains("short-pressed")) {
+                delete elem.dataset.selected;
+            }
         } else {
             elem.classList.toggle("long-pressed");
-            elem.toggleAttribute("data-selected");
-            elem.toggleAttribute("data-recurse");
+            elem.dataset.selected = "";
+            elem.dataset.recurse = "";
         }
 
         elem.dispatchEvent(new Event("longpress", event));