From c2166be06ed1ba038a19e32d2bcdf063d484ac0e Mon Sep 17 00:00:00 2001
From: Anton Volnuhin <anton@volnuhin.com>
Date: Tue, 3 Feb 2026 15:40:54 +0300
Subject: [PATCH] Change multi-select to Cmd+click, Shift+click for interval

- Cmd/Ctrl+click: toggle individual months in/out of selection
- Shift+click: select contiguous interval from current to clicked month

This matches standard selection behavior in file managers and lists.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 app.js | 35 +++++++++++++++++++++++++++++------
 1 file changed, 29 insertions(+), 6 deletions(-)

diff --git a/app.js b/app.js
index d3cbab1..af27c2b 100644
--- a/app.js
+++ b/app.js
@@ -1438,8 +1438,12 @@ async function loadAvailableMonths() {
         btn.appendChild(preview);
         btn.appendChild(label);
         btn.addEventListener('click', (event) => {
-            if (event.shiftKey) {
+            if (event.metaKey || event.ctrlKey) {
+                // Cmd/Ctrl+click: toggle individual month
                 toggleMonthSelection(index);
+            } else if (event.shiftKey) {
+                // Shift+click: select interval from current to clicked
+                selectMonthInterval(index);
             } else {
                 selectSingleMonth(index);
             }
@@ -1456,9 +1460,12 @@ async function loadAvailableMonths() {
     // Set up arrow button handlers
     document.getElementById('prev-month').addEventListener('click', (event) => {
         if (currentMonthIndex > 0) {
-            if (event.shiftKey) {
-                // Add previous month to selection
+            if (event.metaKey || event.ctrlKey) {
+                // Cmd/Ctrl+arrow: toggle previous month
                 toggleMonthSelection(currentMonthIndex - 1);
+            } else if (event.shiftKey) {
+                // Shift+arrow: extend selection to previous month
+                selectMonthInterval(currentMonthIndex - 1);
             } else {
                 selectSingleMonth(currentMonthIndex - 1);
             }
@@ -1467,9 +1474,12 @@ async function loadAvailableMonths() {
 
     document.getElementById('next-month').addEventListener('click', (event) => {
         if (currentMonthIndex < availableMonths.length - 1) {
-            if (event.shiftKey) {
-                // Add next month to selection
+            if (event.metaKey || event.ctrlKey) {
+                // Cmd/Ctrl+arrow: toggle next month
                 toggleMonthSelection(currentMonthIndex + 1);
+            } else if (event.shiftKey) {
+                // Shift+arrow: extend selection to next month
+                selectMonthInterval(currentMonthIndex + 1);
             } else {
                 selectSingleMonth(currentMonthIndex + 1);
             }
@@ -1682,7 +1692,7 @@ function navigateToPath(sunburstData, path) {
     };
 }
 
-// Toggle a month in/out of multi-selection (Shift+click behavior)
+// Toggle a month in/out of multi-selection (Cmd/Ctrl+click behavior)
 async function toggleMonthSelection(index) {
     if (selectedMonthIndices.has(index)) {
         // Don't allow deselecting the last month
@@ -1708,6 +1718,19 @@ async function selectSingleMonth(index) {
     await selectMonth(index);
 }
 
+// Select interval from current month to target (Shift+click behavior)
+async function selectMonthInterval(targetIndex) {
+    const start = Math.min(currentMonthIndex, targetIndex);
+    const end = Math.max(currentMonthIndex, targetIndex);
+
+    selectedMonthIndices.clear();
+    for (let i = start; i <= end; i++) {
+        selectedMonthIndices.add(i);
+    }
+    currentMonthIndex = targetIndex;
+    await renderSelectedMonths();
+}
+
 // Merge transaction data from multiple months
 function mergeMonthsData(monthIndices) {
     const allTransactions = [];