From 35b136944656c9ff9c3b064f47be08224f76c1f9 Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Thu, 24 Sep 2026 18:56:08 +0200 Subject: feat: close the calendar's event pane with Escape Escape was scoped to the pane and only cancelled an edit, so once an event was selected nothing closed the pane again. It is a window shortcut now, labelled Close event: with no edit open it clears the selection and the pane hides. With an edit open it cancels only when focus is inside the pane, so a stray Escape after clicking the grid cannot discard the form. That guard needed the month grid to take focus on a click, which it never did; the view-scoped PgUp/PgDn, Delete and Ctrl+Z were likewise reachable only by Tab until now. Co-Authored-By: Claude Opus 5.5 --- AGENTS.md | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'AGENTS.md') diff --git a/AGENTS.md b/AGENTS.md index 6b69818..0697b20 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -1059,7 +1059,9 @@ scoped to the views with `Qt::WidgetWithChildrenShortcut`.** They are window shortcuts otherwise, and a bare `Delete` or `Ctrl+Z` then fires while the user is typing in the edit form's Title field, deleting the event or undoing the calendar instead of editing text. `EventPane` and the stacked views are the -scopes; Escape is scoped to the pane. +scopes. Escape is the exception and is a WINDOW shortcut, so it closes the +details pane from the grid; with an edit open it cancels only when focus is +inside the pane, so a stray Escape from the grid cannot discard the form. **`calendar_sync_command` is read through a `QStringList` join.** With no quote around the value, QSettings splits a comma-holding string into a `QStringList` -- cgit v1.2.3