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

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:

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