D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Nathaniel Graham
ngraham closed this revision.

REPOSITORY
  R266 Breeze Icons

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

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Nathaniel Graham
ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Perfecto!

REPOSITORY
  R266 Breeze Icons

BRANCH
  master

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

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Phil Stefan
pstefan updated this revision to Diff 41987.
pstefan added a comment.


  Add missing sizes

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15618?vs=41982=41987

BRANCH
  master

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

AFFECTED FILES
  icons-dark/actions/16/document-new.svg
  icons-dark/actions/22/document-new.svg
  icons-dark/actions/24/document-new.svg
  icons-dark/actions/32/document-new.svg
  icons/actions/16/document-new.svg
  icons/actions/22/document-new.svg
  icons/actions/24/document-new.svg
  icons/actions/32/document-new.svg

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Phil Stefan
pstefan added a comment.


  Oops, sorry.
  
  Should be fixed now!

REPOSITORY
  R266 Breeze Icons

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

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Nathaniel Graham
ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  Thanks, this looks perfect for the 16px size! But we'll need the same change 
for the 22, 24, and 23 px sizes too. Could you do those, too?

REPOSITORY
  R266 Breeze Icons

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

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Nathaniel Graham
ngraham added reviewers: Breeze, VDG.
ngraham added a comment.


  Wow, that was fast.

REPOSITORY
  R266 Breeze Icons

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

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Nathaniel Graham
ngraham edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

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

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


D15618: Add plus symbol to document-new.svg

2018-09-20 Thread Phil Stefan
pstefan created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
pstefan requested review of this revision.

REVISION SUMMARY
  Response to bug 398850

TEST PLAN
  Works for me

REPOSITORY
  R266 Breeze Icons

BRANCH
  master

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

AFFECTED FILES
  icons-dark/actions/16/document-new.svg
  icons/actions/16/document-new.svg

To: pstefan
Cc: kde-frameworks-devel, michaelh, ngraham, bruns