Review Request: attempt at panel spacers

[Alessandro Diaferia]

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


How to resize the spacer? :)

- Alessandro


On 2009-04-13 12:22:46, Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/581/
> -----------------------------------------------------------
> 
> (Updated 2009-04-13 12:22:46)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> a thing requested several times and tried to implement several times: an empty space for panels: doing it as a normal applet seems overkill and you can add it on the desktop (ouch)
> this does indeed use an applet (tought about using a simple qgraphicswidget but restoring the config becomes quite more complicate..) but it's private, it's not in the list of applets and the view "knows" the applet, thing that is ugly ugly, but is necessary in order to make impossible to drag it out of the panel
> it's an expanding element and doesn't have config ui at all (and reeeally shouldn't, would be cool to make it possible to be also a configurable fixed ui, but no idea how to make it  not an ui mess)
> 
> things that probably will change:
> the add space action perhaps will become one of the standard panel add space actions
> plasma linking to the applet... uuuugly, drag out of panel avoid perhaps could be made using just the applet name, where is necessary is that now the applet knows when the panel controller is open and it paints instead of being invisible, this is quite nice but the implementation is really evil...
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/CMakeLists.txt 951847 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelappletoverlay.cpp 951847 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelcontroller.h 951847 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelcontroller.cpp 951847 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelspacer.h PRE-CREATION 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelspacer.cpp PRE-CREATION 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/panelview.cpp 951847 
>   /trunk/KDE/kdebase/workspace/plasma/shells/desktop/plasma-applet-panelspacer-internal.desktop PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/581/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco
> 
>