From 5059a7a8b654a8159db85eb5015b51db16ab3502 Mon Sep 17 00:00:00 2001
From: DiTus <ditus@o2.pl>
Date: Thu, 26 Feb 2026 20:36:10 +0100
Subject: [PATCH] Fix duplicate variable declarations in setupEventListeners

Remove duplicate button handler declarations for removeBtn, favoriteBtn, etc.
The duplicate declarations were causing SyntaxError about 'removeBtn' already declared
---
 .../static/js/ui/indicators-panel-new.js      | 23 +++++--------------
 1 file changed, 6 insertions(+), 17 deletions(-)

diff --git a/src/api/dashboard/static/js/ui/indicators-panel-new.js b/src/api/dashboard/static/js/ui/indicators-panel-new.js
index 37a7d1e..e7b68d0 100644
--- a/src/api/dashboard/static/js/ui/indicators-panel-new.js
+++ b/src/api/dashboard/static/js/ui/indicators-panel-new.js
@@ -403,24 +403,13 @@ function setupEventListeners() {
             return;
         }
         
-        // Remove button
-        const removeBtn = e.target.closest('.indicator-btn.remove');
-        if (removeBtn) {
+        // Visibility button (eye)
+        const visibleBtn = e.target.closest('.indicator-btn.visible');
+        if (visibleBtn) {
             e.stopPropagation();
-            const id = removeBtn.dataset.id;
-            if (id && window.removeIndicatorById) {
-                window.removeIndicatorById(id);
-            }
-            return;
-        }
-        
-        // Favorite button
-        const favoriteBtn = e.target.closest('.indicator-btn.favorite');
-        if (favoriteBtn) {
-            e.stopPropagation();
-            const type = favoriteBtn.dataset.type;
-            if (type && window.toggleFavorite) {
-                window.toggleFavorite(type);
+            const id = visibleBtn.dataset.id;
+            if (id && window.toggleIndicatorVisibility) {
+                window.toggleIndicatorVisibility(id);
             }
             return;
         }