Sorry for multple posting, had trobules with reviewboard today. 2010/3/26 Alessandro Diaferia <alediafe...@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
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
