fix(theme): prevent panel element headings from being clipped #334

forbes · 2026-02-26T14:48:19Z

forbes commented

2026-02-26 14:48:19 +00:00

Panel element headings (QGroupBox titles, QSint::ActionGroup headers, QDockWidget title bars) were clipped — only the top ~60% of heading text was visible.

Root Cause

Three QSS issues in KindredCreate.qss:

QGroupBox: margin-top: 12px was insufficient for the title rendered via subcontrol-origin: margin. The title text didn't have enough vertical space in the margin area.
QDockWidget::title: min-height: 18px conflicted with padding: 8px 6px, constraining the content area so text was clipped.
QSint--ActionGroup QToolButton: min-height: 18px forced a minimum height that was then clipped by the C++ setFixedHeight(headerSize) calculation in actionpanelscheme.cpp.

Changes

QGroupBox: Increase margin-top from 12px to 16px
QGroupBox::title: Add 2px vertical padding (padding: 0 4px -> padding: 2px 4px)
QDockWidget::title: Remove min-height: 18px — let Qt auto-size from padding + font metrics
QSint--ActionGroup QToolButton: Set min-height to 0px so the C++ layout controls sizing

Closes #325

Panel element headings (QGroupBox titles, QSint::ActionGroup headers, QDockWidget title bars) were clipped — only the top ~60% of heading text was visible. ## Root Cause Three QSS issues in KindredCreate.qss: 1. **QGroupBox**: `margin-top: 12px` was insufficient for the title rendered via `subcontrol-origin: margin`. The title text didn't have enough vertical space in the margin area. 2. **QDockWidget::title**: `min-height: 18px` conflicted with `padding: 8px 6px`, constraining the content area so text was clipped. 3. **QSint--ActionGroup QToolButton**: `min-height: 18px` forced a minimum height that was then clipped by the C++ `setFixedHeight(headerSize)` calculation in `actionpanelscheme.cpp`. ## Changes - **QGroupBox**: Increase `margin-top` from `12px` to `16px` - **QGroupBox::title**: Add `2px` vertical padding (`padding: 0 4px` -> `padding: 2px 4px`) - **QDockWidget::title**: Remove `min-height: 18px` — let Qt auto-size from padding + font metrics - **QSint--ActionGroup QToolButton**: Set `min-height` to `0px` so the C++ layout controls sizing Closes #325

forbes added 1 commit 2026-02-26 14:48:20 +00:00

fix(theme): prevent panel element headings from being clipped

Build and Test / build (pull_request) Successful in 29m23s

Details

311d911cfa

Three QSS issues caused headings to render with only the top ~60%
visible:

- QGroupBox: margin-top 12px was insufficient for the title rendered
  in subcontrol-origin: margin. Increased to 16px and added 2px
  vertical padding to the title.
- QDockWidget::title: min-height 18px conflicted with padding 8px 6px,
  constraining the content area. Removed min-height to let Qt auto-size
  from padding + font metrics.
- QSint--ActionGroup QToolButton: min-height 18px forced a height that
  was then clipped by the C++ setFixedHeight(headerSize) calculation.
  Set min-height to 0px so the C++ layout controls sizing.

Closes #325

forbes merged commit 4acd09171e into main

2026-02-26 16:34:19 +00:00

forbes deleted branch fix/theme-panel-heading-clipping

2026-02-26 16:34:19 +00:00

forbes referenced this issue from a commit

2026-02-26 16:34:20 +00:00

Merge pull request 'fix(theme): prevent panel element headings from being clipped' (#334) from fix/theme-panel-heading-clipping into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kindred/create#334