[KCM] Slightly improve tab bar's integration with frame

Authored by ngraham on Jul 18 2019, 5:34 PM.



We don't yet have a real QML tab widget (https://bugs.kde.org/show_bug.cgi?id=394296)
so we have to manufacture our own by combining a TabBar with a frame in which the
content sits.

This patch slightly improves the look and feel of our manufactured tab widget
by moving the tab portion down a bit so it overlaps the frame, which does a
better job of mimiching the QWidgets version. It's not perfect, but it's at
least no worse with all the themes I tried, and for some it looks better.

Test Plan

Breeze, before:

Breeze after:

Oxygen, before:

Oxygen, after:

MSWindows, before:

MSWindows, after:

Fusion, before:

Fusion, after:

QtCurve, before:

QtCurve: after:

It's not always perfect, but for some it's better IMO, and it's at least not worse for any.

Diff Detail

R115 Plasma Audio Volume Applet
slightly-better-tab-bar (branched from master)
No Linters Available
No Unit Test Coverage
Build Status
Buildable 14094
Build 14112: arc lint + arc unit
ngraham created this revision.Jul 18 2019, 5:34 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 18 2019, 5:34 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Jul 18 2019, 5:34 PM
ngraham edited the test plan for this revision. (Show Details)Jul 18 2019, 5:37 PM
filipf accepted this revision.Jul 29 2019, 10:46 PM
filipf added a subscriber: filipf.

Looks better. I would also add the bug no. to the comment so all instances of workarounds can be easily grepped when the bug is closed.

This revision is now accepted and ready to land.Jul 29 2019, 10:46 PM
ngraham retitled this revision from [KCM] Slightly imptove tab bar's integration with frame to [KCM] Slightly improve tab bar's integration with frame.Aug 1 2019, 12:59 AM

It's already in a comment above. :)

This revision was automatically updated to reflect the committed changes.