Commit 19d9bc7e authored by Nate Graham's avatar Nate Graham
Browse files

applets/appmenu: fix top-level menu text coloration

28537cf3 made the color dynamic, but
neglected to respect the Plasma color scheme, if any. This causes
problems with Plasma themes that have their own colors and don't
respect the systemwide color scheme, such as Breeze Twilight.

Fix it by using the appropriate colors from the PlasmaCore color scheme
object, not the systemwide object provided by Qt.

BUG: 453348
FIXED-IN: 5.24.6
parent 5a3ef9e8
Pipeline #171843 passed with stage
in 14 minutes and 56 seconds
......@@ -66,9 +66,6 @@ AbstractButton {
contentItem: PC3.Label {
text: controlRoot.Kirigami.MnemonicData.richTextLabel
// Kirigami.Theme.highlightedTextColor returns different colors
// depending on window focus, which does not apply to this applet
// instead, we use palette.highlightedText here, which returns consistent result
color: background.state == MenuDelegate.State.Rest ? palette.windowText : palette.highlightedText
color: background.state == MenuDelegate.State.Rest ? PlasmaCore.Theme.textColor : PlasmaCore.Theme.highlightedTextColor
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment