p1otek/winterfail
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Settings popup close on mobile, remove reset scale button

Browse Source
This commit is contained in:
DiTus
2026-03-21 09:11:35 +01:00
parent 4a3a4a68ce
commit b9928ab88c
2 changed files with 20 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
js/ui/chart.js
View File

@ -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