Re: [PATCH 6/7] add kirigami icons as resource

2016-08-18 Thread Marco Martin
On Thursday 18 August 2016 08:43:59 Martin Gysel wrote: > > 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 onl

Re: [PATCH 6/7] add kirigami icons as resource

2016-08-17 Thread Martin Gysel
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 +> alias="icon

Re: [PATCH 6/7] add kirigami icons as resource

2016-08-16 Thread 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 + alias="icons/folder-sync.svg">breeze-icons/icons/actions/32/folde

[PATCH 6/7] add kirigami icons as resource

2016-08-16 Thread Martin Gysel
--- kirigami-icons.qrc | 13 + kirigami.pri | 1 + 2 files changed, 14 insertions(+) create mode 100644 kirigami-icons.qrc diff --git a/kirigami-icons.qrc b/kirigami-icons.qrc new file mode 100644 index 000..7f3ffa9 --- /dev/null +++ b/kirigami-icons.qrc @@ -0,0 +1,13 @@ +