Commit 91ec99d3 authored by Björn Feber's avatar Björn Feber
Browse files

Make the Notifications KCM category headers look consistent with other Kirigami lists

Summary: {F7172863}

Test Plan: Open the {nav Notifications} KCM and press the {nav Configure...} button.

Reviewers: #plasma, #vdg, ngraham

Reviewed By: #vdg, ngraham

Subscribers: ngraham, plasma-devel, #vdg, #plasma

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D22977
parent 2de7c944
......@@ -108,27 +108,26 @@ Kirigami.Page {
section {
criteria: ViewSection.FullString
property: "sourceType"
delegate: QtControls.ItemDelegate {
delegate: Kirigami.AbstractListItem {
id: sourceSection
width: sourcesList.width
text: {
switch (Number(section)) {
case Private.SourcesModel.ApplicationType: return i18n("Applications");
case Private.SourcesModel.ServiceType: return i18n("System Services");
}
}
// unset "disabled" text color...
contentItem: QtControls.Label {
text: sourceSection.text
// FIXME why does none of this work :(
//Kirigami.Theme.colorGroup: Kirigami.Theme.Active
//color: Kirigami.Theme.textColor
color: rootRow.Kirigami.Theme.textColor
elide: Text.ElideRight
verticalAlignment: Text.AlignVCenter
backgroundColor: Kirigami.Theme.backgroundColor
Kirigami.Theme.inherit: false
Kirigami.Theme.colorSet: Kirigami.Theme.Window
hoverEnabled: false
supportsMouseEvents: false
Kirigami.Heading {
level: 2
text: {
switch (Number(section)) {
case Private.SourcesModel.ApplicationType: return i18n("Applications");
case Private.SourcesModel.ServiceType: return i18n("System Services");
}
}
}
enabled: false
}
}
......
Markdown is supported
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