D15213: Link 16px icon folders to 16@2

2018-09-03 Thread Kai Uwe Broulik
broulik added a comment. Thanks a lot! There's a quirk in Dolphin where it wouldn't pick up the 22px places icons as in `KStandardItemListWidget::pixmapForIcon` it scales the size (22→44) first and then clamps it to an icon size (48) and so it won't find the fixed 22. Also, the problem

D15213: Link 16px icon folders to 16@2

2018-09-03 Thread Chris Rizzitello
rizzitello updated this revision to Diff 40942. rizzitello added a comment. - Link 24 to 24@2x - Link actions/32 to actions/32@2x REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15213?vs=40941=40942 BRANCH 2xIcons REVISION DETAIL

D15213: Link 16px icon folders to 16@2

2018-09-03 Thread Chris Rizzitello
rizzitello updated this revision to Diff 40941. rizzitello added a comment. - Add 22@2x links REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15213?vs=40934=40941 BRANCH 2xIcons REVISION DETAIL https://phabricator.kde.org/D15213 AFFECTED FILES

D15213: Link 16px icon folders to 16@2

2018-09-03 Thread Chris Rizzitello
rizzitello updated this revision to Diff 40934. rizzitello added a comment. - 16@2x icons - icon.theme updated REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15213?vs=40840=40934 BRANCH 16@2x REVISION DETAIL https://phabricator.kde.org/D15213

D15213: Link 16px icon folders to 16@2

2018-09-03 Thread Kai Uwe Broulik
broulik added a comment. About the `index.theme`, what you want to do is, for every folder duplicate the non-2x entry and add a `Scale=2` to it, e.g. [places/16] Size=16 Context=Places Type=Fixed add after it [places/16@2x] Size=16 Scale=2

D15213: Link 16px icon folders to 16@2

2018-09-02 Thread Chris Rizzitello
rizzitello added a comment. In D15213#319039 , @broulik wrote: > I think the suffix should be `@2x`, not just `@2`. Also, I don't think it's neccessary for the `animation` folder. > Anyway, without the `ScalableIcons` entry in `index.theme`

D15213: Link 16px icon folders to 16@2

2018-09-02 Thread Kai Uwe Broulik
broulik added a comment. I think the suffix should be `@2x`, not just `@2`. Also, I don't think it's neccessary for the `animation` folder. Anyway, without the `ScalableIcons` entry in `index.theme` testing this is difficult REPOSITORY R266 Breeze Icons REVISION DETAIL

D15213: Link 16px icon folders to 16@2

2018-09-01 Thread Chris Rizzitello
rizzitello retitled this revision from "link 16 to 16@2" to "Link 16px icon folders to 16@2". rizzitello added reviewers: VDG, Plasma. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D15213 To: rizzitello, #vdg, #plasma Cc: kde-frameworks-devel, michaelh, ngraham,