diff options
| author | Danilo M. <danix@danix.xyz> | 2026-08-15 11:09:29 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-08-15 11:09:29 +0200 |
| commit | b1a7339120385f3fd8f1f5251ec20e3f0ff94b22 (patch) | |
| tree | 18c9adfc3381fdf5ebd09cb405d6ab92da270b0d /src/keymap.h | |
| parent | 51f04ffc3c17da59b6072c4362029c45b780de82 (diff) | |
| download | qtmaildir-b1a7339120385f3fd8f1f5251ec20e3f0ff94b22.tar.gz qtmaildir-b1a7339120385f3fd8f1f5251ec20e3f0ff94b22.zip | |
fix(queries): make a query in the overflow menu runnable
An unpinned query could not be run. Its menu action carried both a triggered
connection and a submenu of edit actions, and Qt does not emit triggered for an
action that owns a menu: clicking the entry only opened the submenu, so the
connection had never fired.
It shipped unnoticed because the menu was the rarely-used half while the user's
queries were pinned buttons. Item 93 moved every query into the menu, which is
how it surfaced, and item 94 makes the menu their only home, so this is now the
path that has to work.
Running is an item inside the submenu, first and above a separator, with the
edit actions below it. The entry keeps its submenu because an unpinned query
must still be editable and deletable.
The test asserts the Run item exists and is first, then that triggering it
reaches the query, then that Edit and Delete survived beside it. Restoring the
old wiring fails it on the first of those, naming the Qt behaviour rather than
just reporting a wrong query string.
Diffstat (limited to 'src/keymap.h')
0 files changed, 0 insertions, 0 deletions
