D28542: Add 16px LibreOffice icons

2020-04-03 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:4cdcd11c07da: Add 16px LibreOffice icons (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28542?vs=79232&id=79251 REVISION DETAIL https:

D28542: Add 16px LibreOffice icons

2020-04-03 Thread Noah Davis
ndavis accepted this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28542 To: manueljlin, #vdg, ngraham, ndavis Cc: ndavis, ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D28542: Add 16px LibreOffice icons

2020-04-03 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a subscriber: ndavis. ngraham added a comment. This revision is now accepted and ready to land. @ndavis, you good with this? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28542 To: manueljlin, #vdg, ngraham Cc: nda

D28542: Add 16px LibreOffice icons

2020-04-03 Thread Manuel Jesús de la Fuente
manueljlin added a comment. Mainly in context menus and small panels. On most apps it isn't really needed, but the LibreOffice icons use 1px lines everywhere and that tends to look quite blurry F8213441: image.png I think it's a good idea to add more

D28542: Add 16px LibreOffice icons

2020-04-03 Thread Nathaniel Graham
ngraham added a comment. LGTM visually. What are these needed for though? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28542 To: manueljlin, #vdg Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D28542: Add 16px LibreOffice icons

2020-04-03 Thread Manuel Jesús de la Fuente
manueljlin retitled this revision from "Adds 16px LibreOffice icons" to "Add 16px LibreOffice icons". REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28542 To: manueljlin, #vdg Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns