diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-07 10:12:10 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-07 10:12:10 +0200 |
| commit | a00d35452fcef8c8bc723ba7045fcae38fe8f315 (patch) | |
| tree | 510d0bb2d9265367648f38bbf43186f471379716 /tests/fixtures/truncated.eml | |
| parent | 6326030b7179580b934ba852b0fd98577bfb464a (diff) | |
| download | qtmaildir-a00d35452fcef8c8bc723ba7045fcae38fe8f315.tar.gz qtmaildir-a00d35452fcef8c8bc723ba7045fcae38fe8f315.zip | |
fix: refresh the current query when the index changes
Item 192's second half, answered by the user: indexing is not repainting.
The sent copy became findable the moment it was sent and a Sent view already
on screen still did not show it, because nothing re-ran the query. The model
cannot insert the row optimistically either, since item 170's constraint
applies: the query never returned that thread.
NotmuchWorker::indexChanged() is emitted at the end of both indexDraftFile()
and removeIndexedFile(), the only two entry points that change what a path
query would return without any query having run. MainWindow connects it to
refreshCurrentQuery(), which covers all three gestures a path view can miss:
a sent copy indexed, a draft saved, a draft's entry dropped on send. Wiring
only the indexing half would have left a ghost draft row visible in a Drafts
view after a send.
The signal carries nothing, so it cannot invite an optimistic insert. It is
emitted after the database closes, so a refresh reaching notmuch on the next
turn of the event loop cannot race the write handle. refreshCurrentQuery()
rather than runCurrentQuery(): a send must not clear the selection, the
expansions or the undo stack of the window behind the composer.
aSentMessageAppearsInASentViewAlreadyOnScreen drives a real send through a
worker-backed window, asserts the Sent view is empty first, and asserts the
row arrives with no second returnPressed() and no sync. Mutation-checked by
disabling the connection.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FWeBw3UqxpBktSc1AkZ6ir
Diffstat (limited to 'tests/fixtures/truncated.eml')
0 files changed, 0 insertions, 0 deletions
