Skip to content
Snippets Groups Projects

qml: make header color transparent by default in `TableViewExt`

Having a header color by default does not make sense here:

  • There is header background color but not background color for the rest.
  • Views are not controls, and are often placed within controls where background is handled properly.
  • A purely transparent rectangle does not create a scene graph node thus not rendered. This is considered an optimization for rendering. In this case, since the background is likely to be the same as the background color of the parent control, the area is painted twice with the same color. If the color needs to be different, the property can be overridden to deviate from transparent color.

Having header color set by default also causes this:

image

Request review @chub.

Merge request reports

Merge request pipeline #577192 passed

Merge request pipeline passed for 567be206

Test coverage 17.90% (-0.77%) from 1 job
Approval is optional

Merged by Steve LhommeSteve Lhomme 3 weeks ago (Mar 16, 2025 12:15pm UTC)

Merge details

  • Changes merged into with 567be206.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #577198 passed with warnings

Pipeline passed with warnings for 567be206 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading