----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128418/ -----------------------------------------------------------
(Updated Aug. 20, 2016, 1:12 p.m.) Status ------ This change has been marked as submitted. Review request for Amarok. Changes ------- Submitted with commit 9449ec56d1fd77a0bfa1f7134a2bf4f91ffd14d3 by Mat?j Laitl on behalf of Antonio Rojas to branch master. Bugs: 365275 https://bugs.kde.org/show_bug.cgi?id=365275 Repository: amarok Description ------- According to the MPRIS2 spec, the DesktopEntry value should be "the basename of the .desktop file" [1], not the desktop-file-id. So for amarok it should be "amarok" and not "kde4-amarok", regardless of where the desktop file is installed. This fixes displaying MPRIS controls in the Amarok taskbar tooltip in Plasma 5.7 [1] https://specifications.freedesktop.org/mpris-spec/latest/Media_Player.html#Property:DesktopEntry Diffs ----- src/dbus/mpris2/MediaPlayer2.cpp cbe1a2a Diff: https://git.reviewboard.kde.org/r/128418/diff/ Testing ------- Player controls are displayed again in the Amarok taskbar tooltip in Plasma 5.7. Thanks, Antonio Rojas