From b9928ab88c800e286cdecc8daa176e5c1f9de899 Mon Sep 17 00:00:00 2001
From: DiTus <ditus@o2.pl>
Date: Sat, 21 Mar 2026 09:11:35 +0100
Subject: [PATCH] fix: Settings popup close on mobile, remove reset scale
 button

---
 index.html     | 13 +++----------
 js/ui/chart.js | 19 +++++++++++++++++--
 2 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/index.html b/index.html
index 3004d44..5e1b2d3 100644
--- a/index.html
+++ b/index.html
@@ -114,16 +114,9 @@
                          <span class="material-symbols-outlined text-sm">fit_screen</span>
                       </button>
                       
-                      <!-- Settings Popup -->
-                     <div class="hidden absolute bottom-10 right-0 bg-[#1a2333] border border-[#2d3a4f] rounded-lg py-2 z-50 w-64 shadow-xl text-sm" id="settingsPopup">
-                            <!-- Reset Scale -->
-                            <div class="px-4 py-2 hover:bg-[#252f3f] cursor-pointer flex items-center gap-3" onclick="window.dashboard.chart.timeScale().fitContent()">
-                                <span class="material-symbols-outlined text-sm">refresh</span> Reset price scale
-                            </div>
-
-                            <hr class="border-[#2d3a4f] my-1">
-
-                            <!-- Scale Toggles -->
+                     <!-- Settings Popup -->
+                      <div class="hidden absolute bottom-10 right-0 bg-[#1a2333] border border-[#2d3a4f] rounded-lg py-2 z-50 w-64 shadow-xl text-sm" id="settingsPopup">
+                             <!-- Scale Toggles -->
                             <div class="px-4 py-2 hover:bg-[#252f3f] cursor-pointer" onclick="window.toggleScaleOption('autoScale')">
                                 <span id="autoScaleCheck">✓</span> Auto (fits data to screen)
                             </div>
diff --git a/js/ui/chart.js b/js/ui/chart.js
index 3a5978c..7a59973 100644
--- a/js/ui/chart.js
+++ b/js/ui/chart.js
@@ -506,14 +506,29 @@ constructor() {
         if (btnSettings && settingsPopup) {
             btnSettings.addEventListener('click', (e) => {
                 e.stopPropagation();
-                settingsPopup.classList.toggle('hidden');
+                settingsPopup.classList.remove('hidden');
+                settingsPopup.classList.add('show');
             });
             
+            // Close popup when clicking outside on desktop
             document.addEventListener('click', (e) => {
-                if (!settingsPopup.contains(e.target) && e.target !== btnSettings) {
+                if (!settingsPopup.contains(e.target) && !btnSettings.contains(e.target)) {
                     settingsPopup.classList.add('hidden');
+                    settingsPopup.classList.remove('show');
                 }
             });
+            
+            // Close popup when clicking on settings items (mobile)
+            settingsPopup.addEventListener('click', (e) => {
+                // Don't close if clicking on input elements or their containers
+                if (e.target.tagName === 'INPUT' || e.target.tagName === 'SELECT' || e.target.tagName === 'BUTTON' || e.target.closest('input') || e.target.closest('select') || e.target.closest('button')) {
+                    e.stopPropagation();
+                    return;
+                }
+                // Close on click inside popup (for mobile touch)
+                settingsPopup.classList.add('hidden');
+                settingsPopup.classList.remove('show');
+            });
         }
         
         // Initialize state from storage