diff options
| author | Danilo M. <danix@danix.xyz> | 2026-08-20 12:38:00 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-08-20 12:38:00 +0200 |
| commit | dca7818517b4444cac5a7c27feb9ff8791e96e8f (patch) | |
| tree | 836fdb9f2b72f84ce144497a22e9ca97891af00a /src/querycompleter.h | |
| parent | fd3bd06580f6bde5d57a68b82cac46eefd89f412 (diff) | |
| download | qtmaildir-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/querycompleter.h')
0 files changed, 0 insertions, 0 deletions
