D11056: [mediacontroller applet] Use media playback status icons

This revision was automatically updated to reflect the committed changes. Closed by commit R120:04dc4c9d2217: [mediacontroller applet] Use media playback status icons (authored by romangg). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE

D11056: [mediacontroller applet] Use media playback status icons

romangg updated this revision to Diff 28906. romangg added a comment. - Also change tooltip icon REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11056?vs=28902=28906 BRANCH mediaAppletStatusIcon REVISION DETAIL

D11056: [mediacontroller applet] Use media playback status icons

broulik added a comment. Can you please also update the tooltip REPOSITORY R120 Plasma Workspace BRANCH mediaAppletStatusIcon REVISION DETAIL https://phabricator.kde.org/D11056 To: romangg, #plasma, andreask, broulik Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg,

D11056: [mediacontroller applet] Use media playback status icons

romangg updated this revision to Diff 28902. romangg added a comment. Rebase on master REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11056?vs=28713=28902 BRANCH mediaAppletStatusIcon REVISION DETAIL https://phabricator.kde.org/D11056

D11056: [mediacontroller applet] Use media playback status icons

andreask accepted this revision. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace BRANCH mediaAppletStatusIcon REVISION DETAIL https://phabricator.kde.org/D11056 To: romangg, #plasma, andreask Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed,

D11056: [mediacontroller applet] Use media playback status icons

romangg added a dependency: D11063: [mediacontroller applet] Reorder elements. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D11056 To: romangg, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D11056: [mediacontroller applet] Use media playback status icons

romangg added a comment. Should we add a fallback mechanism in case the icon pack does not contain the media playback state icons? I mean as a fallback the normal media-playback-start and pause icons again as it is now. REPOSITORY R120 Plasma Workspace REVISION DETAIL

D11056: [mediacontroller applet] Use media playback status icons

romangg edited the summary of this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D11056 To: romangg, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D11056: [mediacontroller applet] Use media playback status icons

romangg edited the summary of this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D11056 To: romangg, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D11056: [mediacontroller applet] Use media playback status icons

romangg added a dependency: D11054: Add media status icons. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D11056 To: romangg, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D11056: [mediacontroller applet] Use media playback status icons

romangg created this revision. romangg added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. romangg requested review of this revision. REVISION SUMMARY Instead of using the action icons to indicate the current state