aboutsummaryrefslogtreecommitdiffstats
path: root/src/threadlistmodel.cpp
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-08-20 12:38:00 +0200
committerDanilo M. <danix@danix.xyz>2026-08-20 12:38:00 +0200
commitdca7818517b4444cac5a7c27feb9ff8791e96e8f (patch)
tree836fdb9f2b72f84ce144497a22e9ca97891af00a /src/threadlistmodel.cpp
parentfd3bd06580f6bde5d57a68b82cac46eefd89f412 (diff)
downloadqtmaildir-dca7818517b4444cac5a7c27feb9ff8791e96e8f.tar.gz
qtmaildir-dca7818517b4444cac5a7c27feb9ff8791e96e8f.zip
docs: choose warning surfaces by consequence, item 123
Two corrections from the user, both of which the spec had wrong. A failed sent-copy write was put in the main window's status bar, on the reasoning that the composer closes so the message needs somewhere persistent. Wrong instinct: the fix for "the window is gone" is a dialog, not a quieter surface. It is the one failure here that produces a silent divergence between what the recipient received and what the local archive holds, and nobody discovers that from a line that showed for a few seconds. It gets a modal. A failed autosave stays in the composer but as a persistent banner rather than a status-area line, since the quit path already escalates that state to a dialog and depends on it surviving. Stated as a rule at the head of the section, because the user's point was general: modal for silent divergence, banner for mid-task, status bar only for what is already obvious. Second correction: the composer's busy indicator is not built inline. A second instance of MainWindow's progress-bar-plus-label pairing is where a widget class earns itself, and "this codebase builds small UI inline" describes what the code does rather than justifying repeating it. Item 134 extracts it and converts MainWindow. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01YDq53rMd3AQp7QmcZzpuBM
Diffstat (limited to 'src/threadlistmodel.cpp')
0 files changed, 0 insertions, 0 deletions