IJHack / QtPass / 24993301662

55%

Build Type

push

github

Committed by web-flow

Commit Message

feat: move process output panel to a QDockWidget; add scroll hysteresis (#252) (#1193)

* feat: move process output panel to a QDockWidget; add scroll hysteresis (#252)

Two follow-up polish items I had filed against #252 after #1172/#1192:

1. **Panel placement.** The output panel was attached via
   `statusBar()->addPermanentWidget(...)` — an 80–150 px tall QTextEdit
   sitting inside what is otherwise a thin status row was an unusual
   home. Moves it to a bottom-area `QDockWidget` (`DockWidgetMovable` +
   `DockWidgetFloatable`, `BottomDockWidgetArea | TopDockWidgetArea`).
   That's the conventional spot for an IDE-style output console, gives
   users detach/move for free, and slots cleanly into one of the named
   `QMainWindow` child slots that uic actually lays out (the same
   structural concern that motivated #1192).

   Construction stays programmatic — no `.ui` change. Renames the panel
   builder out of `initStatusBar()` into a dedicated
   `initProcessOutputPanel()` so the status-bar setup is just the
   logo/label bits again.

2. **Auto-scroll hysteresis.** Previously the `valueChanged` handler
   re-armed `m_autoScroll` the instant the scrollbar value hit
   `maximum`, so a brief overshoot during a slider drag silently
   re-enabled auto-scroll without an explicit user signal. Now skip the
   update while `isSliderDown()` and commit on `sliderReleased` instead.
   Wheel and keyboard scroll never set `isSliderDown()`, so they still
   update immediately. The Clear path already re-arms.

## Test plan

- [x] `make distclean && qmake6 && make -j4` — 0 warnings, 0 errors.
- [x] `make check` — all 12 suites green.
- [x] `doxygen Doxyfile` — 0 warnings.
- [x] `clang-format --style=file --dry-run --Werror` on
      mainwindow.{cpp,h} — clean.
- [x] Default state (`showProcessOutput=false`): central UI intact, no
      dock visible.
- [x] `showProcessOutput=true`: dock appears at the bottom across the
      window width with the title "Process Ou... (continued)

Coverage Stats

0 of 23 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

1825 of 6619 relevant lines covered (27.57%)

26.9 hits per line