fix: prevent Enter key from re-running a settled search #479

Merged
guettlibot merged 5 commits from issue-473-search-result-reorder into main 2026-06-07 02:38:31 +00:00
5 Commits
Author SHA1 Message Date
Bot of Thomas Güttler bcac327f0e Merge branch 'main' into issue-473-search-result-reorder 2026-06-07 04:27:13 +02:00
Thomas SharedInbox d64a33f7ef Merge branch 'main' into issue-473-search-result-reorder 2026-06-07 00:30:44 +02:00
Thomas SharedInbox 9eea81632a Merge branch 'main' into issue-473-search-result-reorder 2026-06-07 00:15:40 +02:00
Thomas SharedInboxandClaude Sonnet 4.6 1e4911d323 fix(ci): forward SSH tunnel directly to dagger engine socket
Eliminates the socat bridge dependency by using OpenSSH's built-in
Unix socket forwarding (-L port:socket_path). The dagger user already
owns /run/dagger/engine.sock so no intermediate TCP listener is needed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-06 23:43:18 +02:00
Thomas SharedInboxandClaude Sonnet 4.6 5ae555b51e fix: prevent Enter key from re-running a settled search (#473)
When the user typed a query, onChanged already fired _runSearch and
results settled. Pressing Enter then triggered onSubmitted → a second
IMAP search whose response could arrive in a different order, silently
reordering the visible list so the tile at position 0 no longer
corresponded to the email the user was about to tap.

Fix: onSubmitted now skips _runSearch when results are already present
(_searchResults != null) or a search is in flight (_searchLoading).
Adds a regression test that verifies the list order is unchanged after
pressing Enter on an already-settled search.

Closes #473

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-06 21:51:20 +02:00