D18785: Use different Font Management KCM "All Fonts" icon

[Björn Feber]

This revision was automatically updated to reflect the committed changes.
Closed by commit R119:e5ef9c97e75e: Use different Font Management KCM All 
Fonts icon (authored by GB_2).

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18785?vs=51115=51162

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

AFFECTED FILES
  kcms/kfontinst/kcmfontinst/GroupList.cpp

To: GB_2, #plasma, #vdg, ngraham
Cc: ngraham, #vdg, plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D18785: Use different Font Management KCM "All Fonts" icon

[Björn Feber]

GB_2 updated this revision to Diff 51115.
GB_2 added a comment.


  Use new icon "font"

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D18785?vs=51032=51115

BRANCH
  use-different-font-management-all-fonts-icon (branched from master)

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

AFFECTED FILES
  kcms/kfontinst/kcmfontinst/GroupList.cpp

To: GB_2, #plasma, #vdg
Cc: ngraham, #vdg, plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D18785: Use different Font Management KCM "All Fonts" icon

[Nathaniel Graham]

ngraham added a comment.


  The name makes me think this is not really semantically correct. Maybe we 
could make a symlink just called `font` or something that points to this icon?
  
  Also this icon only has a 24px version so it would look blurry at the more 
common 16px and 24px sizes, as in fact your screenshot shows.

REPOSITORY
  R119 Plasma Desktop

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

To: GB_2, #plasma, #vdg
Cc: ngraham, #vdg, plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D18785: Use different Font Management KCM "All Fonts" icon

[Björn Feber]

GB_2 created this revision.
GB_2 added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
GB_2 requested review of this revision.

REVISION SUMMARY
  Needed for D18702 .
  F6591513: New Font Management All Fonts Icon.png 


TEST PLAN
  Open the Font Management KCM.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  use-different-font-management-all-fonts-icon (branched from master)

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

AFFECTED FILES
  kcms/kfontinst/kcmfontinst/GroupList.cpp

To: GB_2, #plasma, #vdg
Cc: #vdg, plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart