summaryrefslogtreecommitdiffstats
path: root/src/keymap.cpp
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-08-23 21:15:13 +0200
committerDanilo M. <danix@danix.xyz>2026-08-23 21:15:13 +0200
commitfabcf080652c6e5d57bf234be5e100769a9b965b (patch)
tree0de4222c1e2aab58c38d34c9e0e3c37c68298cc8 /src/keymap.cpp
parentc50bea78e036518ce1a2a3eb899bbb5e305affea (diff)
parentddcae8d02ef46db522b3cf6c228196c7a66a6432 (diff)
downloadqtmaildir-fabcf080652c6e5d57bf234be5e100769a9b965b.tar.gz
qtmaildir-fabcf080652c6e5d57bf234be5e100769a9b965b.zip
Merge branch 'compose-and-send': composing and sending mail
Item 123, built over 2026-08-20 to 2026-08-23 in thirteen tasks against docs/superpowers/specs/2026-08-20-compose-and-send-design.md. The application writes mail now. A composer window per message, markdown as the body, drafts autosaving into the account's Maildir, and sending through a per-account command on stdin rather than any network protocol of this program's own. A countdown with an Undo stands between pressing Send and the command running. Two things came in alongside it. The notmuch auto-tagging hooks moved here from the retiring `mailctl` project and learned that mail this application files itself never arrived, so sent mail and drafts stop appearing in the inbox. And the v1/v2 language is retired: semver on the user-visible surface is the rule, and those labels described a split that composing made obsolete. Hand tested against a fake send command rather than a real one, deliberately: New, Reply and Forward all produce correct messages, a forwarded attachment survives intact, and the sent copy is filed. That testing found the two defects fixed on this branch, and both were invisible to the suite: a composer orphaned by quitting the main window, and every sent message tagged `inbox`. Twenty-two defects were found in the plan document's own draft code while building it, which is why CLAUDE.md says to treat every code block in a plan as a draft.
Diffstat (limited to 'src/keymap.cpp')
-rw-r--r--src/keymap.cpp33
1 files changed, 33 insertions, 0 deletions
diff --git a/src/keymap.cpp b/src/keymap.cpp
index 76c6b60..0df8450 100644
--- a/src/keymap.cpp
+++ b/src/keymap.cpp
@@ -54,6 +54,16 @@ QStringList KeyMap::knownActions()
QStringLiteral("spam_thread"),
QStringLiteral("toggle_unread_thread"),
QStringLiteral("flag_thread"),
+ // Compose and send (item 123). save_message deliberately carries no
+ // default chord: since item 132 a shortcut is a chosen subset rather
+ // than a requirement, and writing the raw message to a file is the
+ // rarely-used escape hatch. Menu reachability is the rule that holds.
+ QStringLiteral("compose"),
+ QStringLiteral("reply"),
+ QStringLiteral("reply_all"),
+ QStringLiteral("reply_no_quote"),
+ QStringLiteral("forward"),
+ QStringLiteral("save_message"),
QStringLiteral("focus_query"),
QStringLiteral("complete_query"),
QStringLiteral("save_query"),
@@ -98,6 +108,29 @@ QList<QPair<QString, QString>> KeyMap::defaultBindings()
{ QStringLiteral("Alt+Down"), QStringLiteral("next_thread") },
{ QStringLiteral("Alt+Up"), QStringLiteral("prev_thread") },
{ QStringLiteral("Return"), QStringLiteral("open_thread") },
+ // Compose and send (item 123), listed where the Message menu presents
+ // them: composing sits above organising.
+ //
+ // PROVISIONAL. The user intends to rework the bindings, and
+ // Ctrl+Alt+R for reply_no_quote is an imperfect fit: the Ctrl+Alt tier
+ // elsewhere means a WIDER SCOPE (the five whole-thread actions), not a
+ // variant of the same scope.
+ //
+ // Each was checked against every sequence in this table, not merely
+ // against the lines above it: these sit near the top, so most of the
+ // table is BELOW them, Ctrl+Shift+U and Ctrl+Shift+S among it.
+ // Checking only upwards would miss exactly those. The near misses:
+ // Ctrl+R is restore, Ctrl+A is select_all and Ctrl+Alt+S is
+ // spam_thread, so none of these five is a reuse.
+ //
+ // save_message gets none. Item 132 made a chord a chosen subset rather
+ // than a requirement, and this is the escape hatch nobody presses a
+ // key for.
+ { QStringLiteral("Ctrl+N"), QStringLiteral("compose") },
+ { QStringLiteral("Ctrl+Shift+R"), QStringLiteral("reply") },
+ { QStringLiteral("Ctrl+Shift+A"), QStringLiteral("reply_all") },
+ { QStringLiteral("Ctrl+Alt+R"), QStringLiteral("reply_no_quote") },
+ { QStringLiteral("Ctrl+Shift+F"), QStringLiteral("forward") },
{ QStringLiteral("Ctrl+E"), QStringLiteral("archive") },
// Del FIRST, and the order matters twice over. defaultSequenceFor()
// returns the first match, and sequenceFor() prefers any binding that