Port the Cover dialog from K3ListView and Q3IconView to KListWidget.
Review Request #115957 - Created Feb. 22, 2014 and submitted
Port the Cover dialog from K3ListView and Q3IconView to KListWidget. The Drag'n'drop thing is still to be done, unfortunately.
|Generally if you're commenting out entire blocks of code, it's better to use '#if 0' and terminate with '#endif'. This ...||Michael Pyne|
This seems to work well and the code looks good. I'll note that I'm starting to be of the opinion that this entire feature should be killed off anyways; once I added support for viewing album art embedded into the files themselves I don't think I ever used this code again. For future development it would probably be best to extend our album art code to be able to replace embedded album art in the music files themselves rather than use a separate cover manager, but that's something for a later date.
Generally if you're commenting out entire blocks of code, it's better to use '#if 0' and terminate with '#endif'. This construct is understood by fancier editors like vim and kate, and there's no risk of accidentally having the comment end to early because a '*/' is present somewhere within the block.