D11050: Remove frames around media buttons

2018-03-07 Thread Roman Gilg 
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:812b32fb0559: Remove frames around media buttons 
(authored by romangg).

REPOSITORY
  R242 Plasma Framework (Library)

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D11050?vs=28692=28899

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

AFFECTED FILES
  src/desktoptheme/breeze/icons/media.svgz

To: romangg, #frameworks, #plasma, ngraham
Cc: ngraham, mart, michaelh

D11050: Remove frames around media buttons

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


  Looks good!

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  reworkMediaIcons

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

To: romangg, #frameworks, #plasma, ngraham
Cc: ngraham, mart, michaelh

D11050: Remove frames around media buttons

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


  +1

REPOSITORY
  R242 Plasma Framework (Library)

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

To: romangg, #frameworks, #plasma
Cc: mart, michaelh

D11050: Remove frames around media buttons

2018-03-05 Thread Roman Gilg 
romangg edited the summary of this revision.

REPOSITORY
  R242 Plasma Framework (Library)

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

To: romangg, #frameworks, #plasma
Cc: michaelh

D11050: Remove frames around media buttons

2018-03-05 Thread Roman Gilg 
romangg added a dependency: D11049: Remove frame around media action icons.

REPOSITORY
  R242 Plasma Framework (Library)

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

To: romangg, #frameworks, #plasma
Cc: michaelh

D11050: Remove frames around media buttons

2018-03-05 Thread Roman Gilg 
romangg created this revision.
romangg added reviewers: Frameworks, Plasma.
Restricted Application added a project: Frameworks.
romangg requested review of this revision.

REVISION SUMMARY
  We were inconsistent on that and it can get in way with placement on buttons.
  
  Mirrors a change in Breeze icons.

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  reworkMediaIcons

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

AFFECTED FILES
  src/desktoptheme/breeze/icons/media.svgz

To: romangg, #frameworks, #plasma
Cc: michaelh