----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108316/ -----------------------------------------------------------
(Updated Jan. 11, 2013, 7:01 a.m.) Review request for Plasma. Changes ------- Used QUrl to: 1) make sure file is valid 2) get its valid path Description ------- Audio player controller has the ability to search through Amarok's collection. This patch uses 'arturl' property of Amarok's reply to show album art in KRunner results. Some considerations: arturl sent by Amarok has file:// prefix which QIcon does not understand. I used QString::remove(0,7) to remove the prefix but it seems hackish. Are there better ways of fixing this issue? I get many warnings when I run krunner: "QPixmap: It is not safe to use pixmaps outside the GUI thread". How should I fix it? Diffs (updated) ----- runners/audioplayercontrol/audioplayercontrolrunner.cpp 75afb75 Diff: http://git.reviewboard.kde.org/r/108316/diff/ Testing ------- Works fine if there's some album art. Reverts to audio-x-generic icon if no arturl is defined. Screenshots ----------- http://git.reviewboard.kde.org/r/108316/s/1013/ Thanks, Emil Sedgh
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
