Review Request: Show album art in krunner results.

Emil Sedgh emilsedgh at gmail.com
Fri Jan 11 07:01:16 UTC 2013


-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130111/6421805d/attachment-0001.html>


More information about the Plasma-devel mailing list