From f644f8cb57c592001a155fb0942598f788f59649 Mon Sep 17 00:00:00 2001 From: r-zakarya <82443831+r-zakarya@users.noreply.github.com> Date: Thu, 9 Jul 2026 23:13:46 +0100 Subject: [PATCH] fix(settings): Esc doesn't close Settings while busy overlay is up MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Pressing Esc during a sync/backup closed Settings under the busy overlay. The Esc handler already bails for open modals — added the same bail when #busyOverlay is visible. Co-Authored-By: Claude Opus 4.8 --- js/app.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/js/app.js b/js/app.js index a0e1817..5c3447e 100644 --- a/js/app.js +++ b/js/app.js @@ -8739,6 +8739,8 @@ async function init() { if (e.key !== 'Escape') return; if (!$('#settingsPanel').classList.contains('is-open')) return; if (document.querySelector('.modal:not(.is-hidden)')) return; + // Busy overlay up (sync/backup running) — don't close under it. + if ($('#busyOverlay') && !$('#busyOverlay').classList.contains('is-hidden')) return; // When the search box has focus AND a query, the input's own // handler clears the query (and stops propagation) — let it run. // Capture phase fires document handlers BEFORE element handlers,