Sorry for multple posting, had trobules with reviewboard today.<br><br><div class="gmail_quote">2010/3/26 Alessandro Diaferia <span dir="ltr"><<a href="mailto:alediaferia@gmail.com">alediaferia@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im"><br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
</div><a href="http://reviewboard.kde.org/r/3396/#review4682" target="_blank">http://reviewboard.kde.org/r/3396/#review4682</a><br>
<div class="im">-----------------------------------------------------------<br>
<br>
<br>
I like how the MC looks like this way. Please be sure to fix what i pointed out. Of course i'd also wait for at least Marco's suggestions about this work.<br>
<br>
- Alessandro<br>
<br>
<br>
On 2010-03-25 19:01:32, Christophe Olinger wrote:<br>
><br>
> -----------------------------------------------------------<br>
</div><div class="im">> This is an automatically generated e-mail. To reply, visit:<br>
> <a href="http://reviewboard.kde.org/r/3396/" target="_blank">http://reviewboard.kde.org/r/3396/</a><br>
> -----------------------------------------------------------<br>
><br>
</div><div class="im">> (Updated 2010-03-25 19:01:32)<br>
><br>
><br>
</div><div><div></div><div class="h5">> Review request for Plasma.<br>
><br>
><br>
> Summary<br>
> -------<br>
><br>
> This patch extends the controller applet by having 6 different layout modes which are adapted to what the media center is currently used for, i.e. browsing pictures, playing videos, etc. It sends a signal to the containment with the current mode. The containment then relayouts the other applets and configures them for the current Mode. These modes are defined as enum in the libs.<br>
> *The browser no longer has any controls. Those are now in the controller.<br>
> *The controller also has a show/hide playlist button and a toggle autohide button for itself.<br>
> *The different modes do not have sensible functions yet. I also need to work on configuring the applets for each mode, like telling the browser to hide, or the player to show.<br>
> *The controller is not really beautiful. I want animations for show(hide icons. I want the modeswitch button in a "drawer" perhaps. The toggle buttons need effects.<br>
><br>
><br>
> Diffs<br>
> -----<br>
><br>
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/abstractmediaitemview.cpp 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/mediabrowser.h 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/mediabrowser.cpp 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediacontroller/controller.h 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediacontroller/controller.cpp 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/mediacontainment.h 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/mediacontainment.cpp 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/medialayout.h 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/medialayout.cpp 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/browser.cpp 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/mediacenter.h 1107457<br>
> /trunk/playground/base/plasma/MediaCenterComponents/shells/plasmediacenter/mainwindow.cpp 1107457<br>
><br>
> Diff: <a href="http://reviewboard.kde.org/r/3396/diff" target="_blank">http://reviewboard.kde.org/r/3396/diff</a><br>
><br>
><br>
> Testing<br>
> -------<br>
><br>
> I tested the controller itself. The actual effect on the other applets when changing modes still needs work.<br>
><br>
><br>
> Thanks,<br>
><br>
> Christophe<br>
><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Alessandro Diaferia<br>KDE Developer<br>KDE e.V. member<br><br>