Add 16px LibreOffice icons
ClosedPublic

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

Details

Summary

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


Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
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.