D10620: Improve sidebar header visibility

Nathaniel Graham noreply at phabricator.kde.org
Sun Feb 18 03:17:03 UTC 2018


ngraham created this revision.
ngraham added reviewers: Plasma, Kirigami, mart.
Restricted Application added projects: Plasma, Kirigami.
Restricted Application added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  BUG: 384638
  FIXED-IN: KDE Plasma 5.12.2
  
  System Settings' sidebar headers were implemented as orginary Kirigami list items that had `enabled: false` set, which made them turn an almost unreadable light color. This was the wrong hack.
  
  This patch changes the hack to a different one that works much better. Ideally we would use real Kirigami list headers here, but the current system settings implementation seems to make this a challenge and I'm not sure it's worth it.

TEST PLAN
  Before:
  
  After, Breeze light:
  
  After, Breeze dark:

REPOSITORY
  R124 System Settings

BRANCH
  more-visible-headers (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D10620

AFFECTED FILES
  sidebar/package/contents/ui/CategoriesPage.qml

To: ngraham, #plasma, #kirigami, mart
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180218/04fdcc41/attachment.html>


More information about the Plasma-devel mailing list