D13137: Updated handles and overflow-menu-*

2018-05-31 Thread Marco Martin 
mart closed this revision.

REPOSITORY
  R266 Breeze Icons

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

To: alex-l, mart, andreask, andreaska, hein, ngraham
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D13137: Updated handles and overflow-menu-*

2018-05-29 Thread Marco Martin 
mart added a comment.


  In D13137#269807 , @ngraham wrote:
  
  > Still not totally thrilled about `overflow-menu-left` and 
`overflow-menu-right`, but I'm willing to give 'em a shot and see how it goes. 
All the other ones are fantastic IMHO. Let's ship it!
  
  
  to me, they should probably keep the old names, handle-right and handle-left, 
if nothing else to not make names in breeze icons explode, having icons left in 
there that are on all effects, deprecated

REPOSITORY
  R266 Breeze Icons

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

To: alex-l, mart, andreask, andreaska, hein, ngraham
Cc: ngraham, kde-frameworks-devel, michaelh, bruns

D13137: Updated handles and overflow-menu-*

2018-05-28 Thread Eike Hein 
hein accepted this revision.

REPOSITORY
  R266 Breeze Icons

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

To: alex-l, mart, andreask, andreaska, hein
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D13137: Updated handles and overflow-menu-*

2018-05-28 Thread Andreas Kainz 
andreask accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R266 Breeze Icons

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

To: alex-l, mart, andreask, andreaska
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D13137: Updated handles and overflow-menu-*

2018-05-28 Thread Marco Martin 
mart added a comment.


  +1 from me, exactly what was needed!
  any definitive ship it from other vdg people

REPOSITORY
  R266 Breeze Icons

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

To: alex-l, mart, andreask, andreaska
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D13137: Updated handles and overflow-menu-*

2018-05-26 Thread Alessandro Longo 
alex-l edited the summary of this revision.

REPOSITORY
  R266 Breeze Icons

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

To: alex-l, mart, andreask, andreaska
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D13137: Updated handles and overflow-menu-*

2018-05-26 Thread Alessandro Longo 
alex-l created this revision.
alex-l added reviewers: mart, andreask, andreaska.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
alex-l requested review of this revision.

REVISION SUMMARY
  - Added the icons in the image as 22px in both Breeze and Breeze Dark, 
support for system color scheme included, SVGs cleaned.
  
  - Now handle-{right;left} symlink point to overflow-menu-{right;left}. In 
future these symlink could be removed or replaced with different icons without 
touching the overflow-menu ones.

REPOSITORY
  R266 Breeze Icons

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

AFFECTED FILES
  icons-dark/actions/22/drag-surface.svg
  icons-dark/actions/22/handle-left.svg
  icons-dark/actions/22/handle-move.svg
  icons-dark/actions/22/handle-right.svg
  icons-dark/actions/22/handle-sort.svg
  icons-dark/actions/22/overflow-menu-left.svg
  icons-dark/actions/22/overflow-menu-right.svg
  icons/actions/22/drag-surface.svg
  icons/actions/22/handle-left.svg
  icons/actions/22/handle-move.svg
  icons/actions/22/handle-right.svg
  icons/actions/22/handle-sort.svg
  icons/actions/22/overflow-menu-left.svg
  icons/actions/22/overflow-menu-right.svg

To: alex-l, mart, andreask, andreaska
Cc: kde-frameworks-devel, michaelh, ngraham, bruns