cullmann added a comment.

  >> For the other icons, I see, there are even more in other subdirs:
  >> 
  >>  cullmann  …  kde  src  breeze-icons  find . -name "kate.svg"         
                                                                                
                                                               master 
  >>  ./icons/apps/32/kate.svg
  >>  ./icons/apps/64/kate.svg
  >>  ./icons/apps/16/kate.svg
  >>  ./icons/apps/22/kate.svg
  >>  ./icons/apps/48/kate.svg
  >>  ./icons-dark/apps/32/kate.svg
  >>  ./icons-dark/apps/64/kate.svg
  >>  ./icons-dark/apps/16/kate.svg
  >>  ./icons-dark/apps/22/kate.svg
  >>  ./icons-dark/apps/48/kate.svg
  >> 
  >> What to do with them?
  > 
  > Since there are already versions of the old icon in multiple sizes, we have 
two options:
  > 
  > 1. Replace them all with appropriate versions for those sizes, which may be 
easy seeing as @tysontan already made them
  > 2. Remove them all.
  
  The smaller versions Tyson has submitted in the task look all like just PNG 
exports of the SVG.
  (I generated them via ksvgtopng in kate.git, see 
https://invent.kde.org/kde/kate/tree/master/kate/icons )
  
  I think for breeze we can just remove the extra SVGs as the new icons (IMHO) 
scales well.

REPOSITORY
  R266 Breeze Icons

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

To: cullmann, ngraham, #vdg, ndavis, tysontan
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to