Review Request: Plasma Mediacenter: Inroduce displaying and modifying tags

Markus Slopianka kamikazow at web.de
Tue Jun 8 00:45:41 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4252/#review6028
-----------------------------------------------------------


Correct me if I missed anything, but as I skimmed through the diff, it looked to me that this is only about Nepomuk tags. Correct?
I may be splitting hairs here, but when I hear "tag editing" in the context of media players, I think of ID3 tags and such.

- Markus


On 2010-06-07 21:43:23, Christophe Olinger wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4252/
> -----------------------------------------------------------
> 
> (Updated 2010-06-07 21:43:23)
> 
> 
> Review request for Plasma and Alessandro Diaferia.
> 
> 
> Summary
> -------
> 
> This patch introduces a new label in the bottom bar. When playing/looking at a media file, it displays its tags. When clicking on the label, the user can add/edit tags to an item. When multiple items are selected, tags are applied to all of them.
> 
> P.S. The videostate.cpp contains comments.
> 
> 
> Diffs
> -----
> 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/mediacenterstate.h 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/mediacenterstate.cpp 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/musicstate.h 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/musicstate.cpp 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/picturestate.h 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/picturestate.cpp 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/videostate.h 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/videostate.cpp 1135632 
>   trunk/playground/base/plasma/MediaCenterComponents/shells/plasmediacenter/mainwindow.cpp 1135632 
> 
> Diff: http://reviewboard.kde.org/r/4252/diff
> 
> 
> Testing
> -------
> 
> Adding, editing tags was tested extensively. No more bugs should remain (TM)
> 
> 
> Thanks,
> 
> Christophe
> 
>



More information about the Plasma-devel mailing list