ngraham created this revision.
ngraham added reviewers: Frameworks, cfeck.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  The permissions editor was using custom low-resolution pixmaps, which did not 
respect
  the icon theme or color scheme and frequently just looked bad, including with 
a high
  DPI screen or when using a dark theme.
  
  This patch ports it to use icons from the active icon theme using 
`QIcon::fromTheme()`.
  
  BUG: 407662
  FIXED-IN: 5.62

TEST PLAN
  All functionality still works.
  
  Looks better, especially with Breeze icons, a dark color scheme, and a high 
DPI scale
  factor: F7144856: Looks good with Breeze dark.png 
<https://phabricator.kde.org/F7144856>

REPOSITORY
  R241 KIO

BRANCH
  port-to-qicon (branched from master)

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

AFFECTED FILES
  src/widgets/images/group-grey.png
  src/widgets/images/group.png
  src/widgets/images/mask.png
  src/widgets/images/others-grey.png
  src/widgets/images/others.png
  src/widgets/images/user-grey.png
  src/widgets/images/user.png
  src/widgets/images/yes.png
  src/widgets/images/yespartial.png
  src/widgets/kacleditwidget.cpp
  src/widgets/kacleditwidget_p.h

To: ngraham, #frameworks, cfeck
Cc: kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns

Reply via email to