---
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
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel