D21218: Use Anjuta's icon for Knights, since it works and Anjuta is abandoned at this point

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


  New Knights icon based on Anjuta's

REPOSITORY
  R266 Breeze Icons

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

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
Cc: ndavis, ognarb, kde-frameworks-devel, michaelh, ngraham, bruns

D21218: Use Anjuta's icon for Knights, since it works and Anjuta is abandoned at this point

2019-05-14 Thread Noah Davis 
ndavis added a comment.


  In D21218#465340 , @ognarb wrote:
  
  > Why not mirror the Anjuta icon and use the same background color as the 
original Knights icon? So that it looks closer to the original.
  
  
  I think this is probably better and might prevent confusion for existing 
Knights users.

REPOSITORY
  R266 Breeze Icons

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

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

D21218: Use Anjuta's icon for Knights, since it works and Anjuta is abandoned at this point

2019-05-14 Thread Carl Schwan 
ognarb added a comment.


  Why not mirror the Anjuta icon and use the same background color as the 
original Knights icon? So that it looks closer to the original.

REPOSITORY
  R266 Breeze Icons

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

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

D21218: Use Anjuta's icon for Knights, since it works and Anjuta is abandoned at this point

2019-05-14 Thread Nathaniel Graham 
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: kde-frameworks-devel, michaelh, ngraham, bruns

D21218: Use Anjuta's icon for Knights, since it works and Anjuta is abandoned at this point

2019-05-14 Thread Nathaniel Graham 
ngraham created this revision.
ngraham added a reviewer: VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  Anjuta's icon depicts a knight-looking horse. However Anjuta is unmaintained 
and abandoned,
  the last release having been seven years ago. So let's use its icon for 
Knights, which
  needs one.
  
  CCBUG: 407527

REPOSITORY
  R266 Breeze Icons

BRANCH
  anjuta-to-knights (branched from master)

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

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

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