IJHack / QtPass / 27501799111

57%

Build Type

push

github

Committed by web-flow

Commit Message

fix: install event filter once and add UI watchdog (#1512) (#1536)

Third slice of the MainWindow decomposition (#1512), the UiState concern.
Per the issue's "or kill it" option this fixes the two real problems in the
setUiElementsEnabled hammer rather than wrapping it in a new class:

- The search-box key event filter was re-installed on every
  setUiElementsEnabled call; install it once in the constructor instead.
- Add a 30s single-shot watchdog: setUiElementsEnabled(false) arms it and
  setUiElementsEnabled(true) disarms it, so if a backend operation disables
  the UI but never signals completion the interface re-enables itself
  instead of staying stuck.

No behavioural change in the normal path. mainwindow tests pass; doxygen
and reuse clean.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Coverage Stats

7 of 10 new or added lines in 1 file covered. (70.0%)

379 existing lines in 3 files now uncovered.

3977 of 6925 relevant lines covered (57.43%)

36.21 hits per line