D18379: Add tools icon

2019-01-19 Thread Noah Davis
This revision was automatically updated to reflect the committed changes. Closed by commit R266:1000c8b4884e: Add tools icon (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18379?vs=49862&id=49884 REVISION DETAIL https://phabricator

D18379: Add tools icon

2019-01-19 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. In D18379#396445 , @ndavis wrote: > In D18379#396432 , @ngraham wrote: > > > Very nice! Just what I had in mind. Do we really need a 32px vers

D18379: Add tools icon

2019-01-19 Thread Noah Davis
ndavis added a comment. In D18379#396432 , @ngraham wrote: > Very nice! Just what I had in mind. Do we really need a 32px version though? A number of icons start with `tools-`, which means that this icon is the last fallback option. It co

D18379: Add tools icon

2019-01-19 Thread Nathaniel Graham
ngraham added a comment. Very nice! Just what I had in mind. Do we really need a 32px version though? I thought we were trying to move towards all ≥ 32px icons being colorful, and not using the 32px size for action icons? REPOSITORY R266 Breeze Icons BRANCH tools-icon (branched from mas

D18379: Add tools icon

2019-01-19 Thread Björn Feber
GB_2 accepted this revision. GB_2 added a comment. This revision is now accepted and ready to land. Very nice! REPOSITORY R266 Breeze Icons BRANCH tools-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D18379 To: ndavis, #vdg, GB_2 Cc: GB_2, rooty, kde-framework

D18379: Add tools icon

2019-01-19 Thread Root
rooty added a comment. nice, +1 REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18379 To: ndavis, #vdg Cc: rooty, kde-frameworks-devel, michaelh, ngraham, bruns

D18379: Add tools icon

2019-01-19 Thread Noah Davis
ndavis updated this revision to Diff 49862. ndavis added a comment. Fix messed up breeze-dark 32px version REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18379?vs=49861&id=49862 BRANCH tools-icon (branched from master) REVISION DETAIL https://pha

D18379: Add tools icon

2019-01-19 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18379 To: ndavis, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18379: Add tools icon

2019-01-19 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18379 To: ndavis, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18379: Add tools icon

2019-01-19 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18379 To: ndavis, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18379: Add tools icon

2019-01-19 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REVISION SUMMARY Adds a `tools` icon for menus that show a list of tools/programs (e.g., the "Tools" menus i