-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126647/
-----------------------------------------------------------

(Updated Jan. 19, 2016, 9:47 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma, KDE Usability and Eike Hein.


Changes
-------

Submitted with commit f9e87f911bb7823b1ae1aca16606cc9b50adf7e0 by Kai Uwe 
Broulik to branch master.


Bugs: 352126
    https://bugs.kde.org/show_bug.cgi?id=352126


Repository: plasma-desktop


Description
-------

This adds media player controls to the tooltip of a media player, using the 
already existing mpris dataengine this was pretty straightforward to implement.

When album art is available, no window thumbnails will be shown, instead the 
album art. There will be a close button on the album art closing the first 
window. Multiple windows for a player is uncommon and you can still access them 
all by clicking the task manager entry.

Interestingly enough, Amarok does not expose its album art through MPris. Also, 
if it wouldn't crash whilst doing so, you could control Amarok when it's main 
window is closed if you have a launcher pinned to your task manager, basically 
rendering its tray icon obsolete.


Diffs
-----

  applets/taskmanager/package/contents/ui/Task.qml 2a88886 
  applets/taskmanager/package/contents/ui/ToolTipDelegate.qml 4bf0b31 

Diff: https://git.reviewboard.kde.org/r/126647/diff/


Testing
-------

Works with VLC and Audacious, does not work with Dragon as the player announces 
itself as "dragonplayer" while its Desktop file says "dragon"


File Attachments
----------------

VLC with album art
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/05/3058dacb-1dfd-464d-a1ec-be90bc9e58a8__mpristaskmanagerreview1.png
Amarok without album art
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/05/6799d6f6-d505-4f56-9531-3013a3e34ae6__mpristaskmanagerreview2.png
No gradient
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/06/25298761-28bb-4620-b768-dd0259f19b82__mpristaskmanagerreview3.png
Raising Amarok through its launcher
  
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/07/8dfcb9b6-6e11-4017-b4fc-70d92c9ded2a__mpristaskmanagerreview4.png


Thanks,

Kai Uwe Broulik

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to