From f40024cc6727a9856581666f0cd1261560a0b4b3 Mon Sep 17 00:00:00 2001
From: Zaki <18zaki18@gmail.com>
Date: Sat, 9 May 2026 18:46:27 +0100
Subject: [PATCH] Fix: mousedown clears selectedIds before batch callback runs

Document mousedown handler was clearing selectedIds when clicking 'Yes' (button element). Since mousedown fires before click, selectedIds was empty by the time the async callback read it, causing no-op deletes but still showing the toast. Added .batch-confirm-overlay to the mousedown exclusion list so clicking the confirm dialog won't trigger clearSelection().
---
 js/app.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/app.js b/js/app.js
index e240198..b5103f1 100644
--- a/js/app.js
+++ b/js/app.js
@@ -1306,7 +1306,7 @@ document.getElementById('entriesContainer').addEventListener('dragstart', functi
 // Document mousedown: rect selection on vault background, clear outside vault
 document.addEventListener('mousedown', function(e) {
     if (e.button !== 0 || rectState.active) return;
-    if (e.target?.closest?.('.entry-card,.entry-row,.entry-compact,.table-row-drag,.detail-card,.detail-nav,#batchBar,.custom-modal-overlay.show,.edit-modal.show,.modal-overlay.show,#genModal,#settingsMenu')) return;
+    if (e.target?.closest?.('.entry-card,.entry-row,.entry-compact,.table-row-drag,.detail-card,.detail-nav,#batchBar,.custom-modal-overlay.show,.edit-modal.show,.modal-overlay.show,#genModal,#settingsMenu,.batch-confirm-overlay')) return;
     if (e.target?.closest?.('button,input,select,.folders-bar,.toolbar,#trashActions,.settings-dropdown,.fab,.auth-section')) {
         if (selectedIds.size > 0) clearSelection();
         return;