Review Request: Plasma Mediacenter: Inroduce displaying and modifying tags

Christophe Olinger olingerc at binarylooks.com
Mon Jun 7 23:43:23 CEST 2010


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

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