Commit 04dc4c9d authored by Roman Gilg's avatar Roman Gilg
Browse files

[mediacontroller applet] Use media playback status icons

Summary:
Instead of using the action icons to indicate the current state use
the new media playback status icons.

This is an alternative solution to the problem that the mediacontroller applet
icon in the system tray and the play/pause controll in the player are always
the opposite.

While it is still the opposite icon there is now a slight visual difference
to show that these icons mean different things.

This is meant as a solution to the problem discussed in D10032.

{F5741436}

{F5741437}

Reviewers: #plasma, andreask, broulik

Reviewed By: #plasma, andreask, broulik

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D11056
parent d511a766
......@@ -62,7 +62,7 @@ Item {
Plasmoid.switchWidth: units.gridUnit * 14
Plasmoid.switchHeight: units.gridUnit * 10
Plasmoid.icon: albumArt ? albumArt : "media-playback-start"
Plasmoid.icon: albumArt ? albumArt : "media-playback-playing"
Plasmoid.toolTipMainText: i18n("No media playing")
Plasmoid.toolTipSubText: ""
Plasmoid.toolTipTextFormat: Text.PlainText
......@@ -143,9 +143,9 @@ Item {
Plasmoid.fullRepresentation: ExpandedRepresentation {}
Plasmoid.compactRepresentation: PlasmaCore.IconItem {
source: root.state === "playing" ? "media-playback-start" :
root.state === "paused" ? "media-playback-pause" :
"media-playback-stop"
source: root.state === "playing" ? "media-playback-playing" :
root.state === "paused" ? "media-playback-paused" :
"media-playback-stopped"
active: compactMouse.containsMouse
MouseArea {
......@@ -250,7 +250,7 @@ Item {
PropertyChanges {
target: plasmoid
icon: albumArt ? albumArt : "media-playback-start"
icon: albumArt ? albumArt : "media-playback-playing"
toolTipMainText: track
toolTipSubText: artist ? i18nc("Artist of the song", "by %1", artist) : ""
}
......@@ -261,7 +261,7 @@ Item {
PropertyChanges {
target: plasmoid
icon: albumArt ? albumArt : "media-playback-pause"
icon: albumArt ? albumArt : "media-playback-paused"
toolTipMainText: track
toolTipSubText: artist ? i18nc("Artist of the song", "by %1 (paused)", artist) : i18n("Paused")
}
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment