Make the theme info QuickWidget transparent
AbandonedPublic

Authored by filipf on Dec 13 2019, 10:52 PM.

Details

Reviewers
davidedmundson
ngraham
Group Reviewers
Plasma
Summary

We're currently using an ugly hack in our main.qml file to make the QuickWidget that shows theme info have it's color the same as the QWidget it's nested in.

The hack is not only ugly but totally breaks apart with any widget style that's not Breeze.

The correct solution has always been to just set the QuickWidget to be transparent. It now blends with every widget style and we don't need a hack anymore.

Test Plan

Example: Oxygen

Before:

After:

Diff Detail

Repository
R123 SDDM Configuration Panel (KCM)
Branch
transparent-quick-widget (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 19819
Build 19837: arc lint + arc unit
filipf created this revision.Dec 13 2019, 10:52 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 13 2019, 10:52 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
filipf requested review of this revision.Dec 13 2019, 10:52 PM
filipf edited the summary of this revision. (Show Details)Dec 13 2019, 10:53 PM
filipf edited the test plan for this revision. (Show Details)
filipf added reviewers: Plasma, davidedmundson.
filipf updated this revision to Diff 71484.Dec 13 2019, 10:54 PM
  • remove old hack comment

Excellent solution. Question: do we even need a Rectangle at all if it's going to be transparent? Can it just be an Item?

filipf updated this revision to Diff 71485.Dec 13 2019, 11:18 PM
  • downgrade Rectangle to Item since we don't need anything fancy anymore (good tip @ngraham)
ngraham accepted this revision.Dec 13 2019, 11:24 PM

Shipit! Stable branch please.

BTW, GridViewKCM, SimpleKCM, and ScrollViewKCM seem like they needs the same treatment, now that I'm navigating System Settings with the Oxygen style to test this.

This revision is now accepted and ready to land.Dec 13 2019, 11:24 PM
filipf added a comment.EditedDec 14 2019, 8:06 AM

There might be an unfortunate side effect: https://forum.qt.io/topic/57634/qt-wa_alwaysstackontop-influences-font-rendering-in-qml/5

Which is acceptable here in my opinion since blending doesn't even exist with Breeze, but could become a problem if present everywhere and in prominent places.

filipf abandoned this revision.Dec 23 2019, 9:38 AM

Going to have to be a no from me since the lack of font anti-aliasing looks pretty bad with darker color schemes. Not sure if it's worth it to report the bug to Qt.