ngraham added a comment.
Thanks, much better now. In D27044#603538 <https://phabricator.kde.org/D27044#603538>, @cullmann wrote: > In D27044#603537 <https://phabricator.kde.org/D27044#603537>, @cullmann wrote: > > > 2 questions: > > > > 1. should I use the same icon for dark? > Yep. > 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. 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
