Commit e87e94e3 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


(cherry picked from commit 19d9bc7e)
parent e2b631d4
Pipeline #171847 passed with stage
in 10 minutes and 49 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