[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Review Request: Plasma-Mediacenter: Cosmetics for the playlist
From:       "Christophe Olinger" <olingerc () binarylooks ! com>
Date:       2010-05-30 16:30:10
Message-ID: 20100530163010.2432.16143 () localhost
[Download RAW message or body]


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

(Updated 2010-05-30 16:30:10.201306)


Review request for Plasma and Alessandro Diaferia.


Changes
-------

The playlist is now positioned correctly no matter which theme is applied


Summary
-------

This (for once very small) patch stretches the playlist fully between the two \
horizontal bars and makes it a bit wider. It also gives us the ability to theme the \
playlist. Actually it looks best without any background when the use is browsing \
things, but a background would be needed when a video is playing and the playlist is \
shown. What do ya think?

Bug: The playlist handler does not correctly follow the resizing of the playlist


Diffs (updated)
-----

  trunk/playground/base/plasma/MediaCenterComponents/applets/mediacontroller/controller.cpp \
1132326   trunk/playground/base/plasma/MediaCenterComponents/applets/mediainfobar/mediainfobar.cpp \
1132326   trunk/playground/base/plasma/MediaCenterComponents/applets/playlist/playlistapplet.h \
1132326   trunk/playground/base/plasma/MediaCenterComponents/applets/playlist/playlistapplet.cpp \
1132326   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/medialayout.h \
1132326   trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/medialayout.cpp \
1132326 

Diff: http://reviewboard.kde.org/r/4167/diff


Testing
-------


Thanks,

Christophe

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]