D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:5a2dfb5804ba: Create new icon for Knights based on 
Anjutas icon (authored by ngraham).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D21218?vs=58105=58109#toc

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D21218?vs=58105=58109

REVISION DETAIL
  https://phabricator.kde.org/D21218

AFFECTED FILES
  icons/apps/48/knights.svg

To: ngraham, #vdg, ndavis
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns


D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Noah Davis
ndavis accepted this revision.
ndavis added a comment.
This revision is now accepted and ready to land.


  LGTM

REPOSITORY
  R266 Breeze Icons

BRANCH
  anjuta-to-knights (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D21218

To: ngraham, #vdg, ndavis
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns


D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Nathaniel Graham
ngraham edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

BRANCH
  anjuta-to-knights (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D21218

To: ngraham, #vdg, ndavis
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns


D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Nathaniel Graham
ngraham added a comment.


  Thanks!

REPOSITORY
  R266 Breeze Icons

BRANCH
  anjuta-to-knights (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D21218

To: ngraham, #vdg, ndavis
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns


D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Nathaniel Graham
ngraham updated this revision to Diff 58105.
ngraham added a comment.


  Optimize with `scour`

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D21218?vs=58096=58105

BRANCH
  anjuta-to-knights (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D21218

AFFECTED FILES
  icons-dark/apps/48/knights.svg
  icons/apps/48/knights.svg

To: ngraham, #vdg, ndavis
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns


D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Noah Davis
ndavis requested changes to this revision.
ndavis added a comment.
This revision now requires changes to proceed.


  Once you've optimized the SVGs, I'll approve this.
  
  
https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#SVG_optimization

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D21218

To: ngraham, #vdg, ndavis
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns


D21218: Create new icon for Knights based on Anjuta's icon

2019-05-14 Thread Nathaniel Graham
ngraham retitled this revision from "Use Anjuta's icon for Knights, since it 
works and Anjuta is abandoned at this point" to "Create new icon for Knights 
based on Anjuta's icon".
ngraham edited the summary of this revision.
ngraham edited the test plan for this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D21218

To: ngraham, #vdg
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns