D6313: Support Icon Scale from Icon naming specification 0.13

[Nathaniel Graham]

ngraham added a comment.


  In D6313#238605 <https://phabricator.kde.org/D6313#238605>, @acrouthamel wrote:
  
  > I think both of us missed this part at the bottom of @broulik's description:
  >
  > > This way you designers can now create dedicated 2x SVGs for those usecases, ie. we can have a 16px icon as well as a 16px at 2x icon rather than it just taking the 32px icon which might not fit. In case a 16px at 2x icon is not present it will load the 32px icon instead as it did before. This way one could even create a high dpi Oxygen theme.
  >
  > So #Breeze <https://phabricator.kde.org/tag/breeze/> will show the hi-res icons like it does now since it doesn't have any @2x folders or symlinks (an example of that can be found in Papirus <https://github.com/PapirusDevelopmentTeam/papirus-icon-theme/tree/master/Papirus>). Moving forward, #Breeze <https://phabricator.kde.org/tag/breeze/> could create these @2x folders, populating them with beautiful Hi-DPI-acceptable icons, or whatever they want, to ensure interface consistency.
  >
  > But for now, it would be status-quo, even with this patch committed. So no one will notice, except those of us with @2x-compatible, 3rd party icon themes right now.
  
  
  Oops, so I did. Let me actually test this before I run my mouth any more...
  
  [goes to apply the patch]
  
  Kai, could you re-base this on master so it applies?

REPOSITORY
  R302 KIconThemes

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

To: broulik, kde-frameworks-devel, #plasma, #vdg
Cc: hein, rkflx, acrouthamel, ngraham, elvisangelaccio, mart, kvermette, cfeck, davidedmundson, plasma-devel, #frameworks, ragreen, Pitel, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180402/ddc1ae49/attachment-0001.html>