D7810: Add missing icon sizes

2017-11-14 Thread mandian
mandian added a comment. Great! Thank you all! REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: ngraham, #breeze, #vdg, jensreuterberg, #frameworks, hein, mandian Cc: andreask, hein, ngraham, cfeck, #frameworks

D7810: Add missing icon sizes

2017-11-13 Thread mandian
mandian added a comment. This is a patch I wrote when I was packaging MATE desktop. What I only care is it works well :). REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze, #vdg, jensreuterberg, #frameworks, ngraham, hein Cc: andreask

D7810: Add missing icon sizes

2017-11-13 Thread mandian
mandian added a comment. In https://phabricator.kde.org/D7810#167373, @ngraham wrote: > You would renounce your copyright in this case. If it is only a matter of copyright yes, of course :). REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810

D7810: Add missing icon sizes

2017-11-13 Thread mandian
mandian added a comment. @ngraham the mail is the one you know but please do not expose it in a public page (I'd like to avoid spam bots). The name I use is the same you know too (I use this to sign the packages I build for OpenMandirva). @jensreuterberg sure! I'll ask for he

D7810: Add missing icon sizes

2017-11-12 Thread mandian
mandian added a comment. Even if simplified icons may look better with small size I did not notice any fuzzy effects in smaller icon. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze, #vdg, jensreuterberg, #frameworks, ngraham, hein Cc

D7810: Add missing icon sizes

2017-11-11 Thread mandian
mandian updated this revision to Diff 22206. mandian added a comment. I cleaned the patch and I included icon-dark too. REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7810?vs=19506&id=22206 REVISION DETAIL https://phabricator.kde.org/D

D7810: Add missing icon sizes

2017-11-11 Thread mandian
mandian added a comment. It like changes for size of icons in MimeTypes is not more needed because the bug is in caja and not in Breeze theme. Do you prefer I submit a new patch without these two changes? INLINE COMMENTS > hein wrote in index.theme:297 > Hmm why lower this one: I di

D7810: Add missing icon sizes

2017-10-10 Thread mandian
mandian requested review of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze, #vdg, jensreuterberg Cc: ngraham, cfeck, #frameworks

D7810: Add missing icon sizes

2017-10-10 Thread mandian
mandian planned changes to this revision. mandian added a comment. Hi ngraham I added some more details and screenshots to test plan. Please let me know is more is needed. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze, #vdg

D7810: Add missing icon sizes

2017-10-10 Thread mandian
mandian edited the summary of this revision. mandian edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze, #vdg, jensreuterberg Cc: ngraham, cfeck, #frameworks

D7810: Add missing icon sizes

2017-09-16 Thread mandian
mandian requested review of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze Cc: #frameworks

D7810: Add missing icon sizes

2017-09-13 Thread mandian
mandian planned changes to this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D7810 To: mandian, #breeze Cc: #frameworks

D7810: Add missing icon sizes

2017-09-13 Thread mandian
mandian created this revision. mandian added a project: Breeze. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Add missing icon sizes TEST PLAN Add missing scalable icon sizes at index.theme. If not all size are