[PATCH 6/7] add kirigami icons as resource

Martin Gysel me at bearsh.org
Thu Aug 18 06:43:59 UTC 2016


Am 16.08.2016 um 21:27 schrieb Marco Martin:
> did some comments inline but would be better if this was done in phabricator
> 
> On Tuesday 16 August 2016 20:23:51 Martin Gysel wrote:
>> alias="icons/document-decrypt.svg">breeze-icons/icons/actions/32/document-d
>> ecrypt.svg</file> +        <file
>> alias="icons/folder-sync.svg">breeze-icons/icons/actions/32/folder-sync.svg
>> </file> +        <file
>> alias="icons/go-next.svg">breeze-icons/icons/actions/24/go-next.svg</file>
> 
> don't use 24 pixels icons, they tend to scale badly on sizes used by the app

so what's the recommendations regarding icon sizes
48px -> 32px -> 22px ? or skip the 22px as well?
go-next.svg, go-next.svg and go-previous.svg are only available in
16px,22px and 24px


More information about the Plasma-devel mailing list