IJHack / QtPass / 24970081225 / 1
55%
main: 55%

DEFAULT BRANCH: main
Ran
Files 72
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 27.593%. Remained the same
24970081225.1

push

github

web-flow 
fix: move processOutputWidget inside centralWidget so it's actually wired up (#1192)

#1172 added the process output panel as a sibling of centralWidget at
the QMainWindow level:

  <widget class="QMainWindow" name="MainWindow">
    <widget class="QWidget" name="centralWidget"> ... </widget>
    <widget class="QWidget" name="processOutputWidget"> ... </widget>  <-- here
    <widget class="QStatusBar" name="statusBar"/>
    <widget class="QToolBar" name="toolBar"> ... </widget>
  </widget>

QMainWindow recognises a fixed set of named slots for its top-level
children — centralWidget, statusBar, menuBar, toolBars and dock widgets.
Anything else parented at that level is created by uic but never placed
in the QMainWindow's layout, so it gets ignored by the QMainWindowLayout
and renders on top of (i.e. obscures) the centralWidget content.

The visible symptom is that with HEAD, the QtPass main window comes up
with the toolbar at top and the status bar at the bottom but a *blank*
centre — no profile selector, no search box, no tree view, no text
browser. Bisected against pre-#1172 (`8e257d633`) where the same
.ui structure (modulo this addition) renders correctly.

Move processOutputWidget to row=2/colspan=2 inside centralWidget's
QGridLayout (where it's a real layout participant) and let it stretch
across the bottom of the central area. The runtime call to
`statusBar()->addPermanentWidget(ui->processOutputWidget)` from
initStatusBar() then re-parents it to the status bar; at that point
QMainWindow's layout reclaims the row 2 space for the actual content
above it.

Verified by screenshot: search box, tree view, profile selector and
welcome browser all reappear; status bar with the lock icon still
shows; no test regressions (12/12 suites green); doxygen clean.

1825 of 6614 relevant lines covered (27.59%)

26.92 hits per line

Source Files on job 24970081225.1