Add 16px LibreOffice icons

Authored by manueljlin on Apr 3 2020, 3:58 PM.



Sorry if I did something wrong, I'm still figuring this stuff out

Diff Detail

R266 Breeze Icons
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
manueljlin requested review of this revision.Apr 3 2020, 3:58 PM
manueljlin created this revision.
manueljlin edited the summary of this revision. (Show Details)Apr 3 2020, 4:01 PM
manueljlin added a reviewer: VDG.
manueljlin set the repository for this revision to R266 Breeze Icons.
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 3 2020, 4:01 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
manueljlin retitled this revision from Adds 16px LibreOffice icons to Add 16px LibreOffice icons.Apr 3 2020, 4:03 PM
ngraham added a subscriber: ngraham.Apr 3 2020, 4:23 PM

LGTM visually. What are these needed for though?

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

I think it's a good idea to add more variations to breeze icons in general to make icons look sharp everywhere

ngraham accepted this revision.Apr 3 2020, 6:56 PM
ngraham added a subscriber: ndavis.

@ndavis, you good with this?

This revision is now accepted and ready to land.Apr 3 2020, 6:56 PM
ndavis accepted this revision.Apr 3 2020, 9:13 PM
This revision was automatically updated to reflect the committed changes.