Review Request: Initial work on the flexible controller of the Plasma Media Center

[Alessandro Diaferia]

Sorry for multple posting, had trobules with reviewboard today.

2010/3/26 Alessandro Diaferia <alediaferia at gmail.com>

>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3396/#review4682
> -----------------------------------------------------------
>
>
> 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.
>
> - Alessandro
>
>
> On 2010-03-25 19:01:32, Christophe Olinger wrote:
> >
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://reviewboard.kde.org/r/3396/
> > -----------------------------------------------------------
> >
> > (Updated 2010-03-25 19:01:32)
> >
> >
> > Review request for Plasma.
> >
> >
> > Summary
> > -------
> >
> > 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.
> > *The browser no longer has any controls. Those are now in the controller.
> > *The controller also has a show/hide playlist button and a toggle
> autohide button for itself.
> > *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.
> > *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.
> >
> >
> > Diffs
> > -----
> >
> >
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/abstractmediaitemview.cpp
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/mediabrowser.h
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/mediabrowser.cpp
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediacontroller/controller.h
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/applets/mediacontroller/controller.cpp
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/mediacontainment.h
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/mediacontainment.cpp
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/medialayout.h
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/containments/mediacontainment/medialayout.cpp
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/browser.cpp
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/libs/mediacenter/mediacenter.h
> 1107457
> >
> /trunk/playground/base/plasma/MediaCenterComponents/shells/plasmediacenter/mainwindow.cpp
> 1107457
> >
> > Diff: http://reviewboard.kde.org/r/3396/diff
> >
> >
> > Testing
> > -------
> >
> > I tested the controller itself. The actual effect on the other applets
> when changing modes still needs work.
> >
> >
> > Thanks,
> >
> > Christophe
> >
> >
>
>


-- 
Alessandro Diaferia
KDE Developer
KDE e.V. member
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100326/dd610422/attachment.htm