Review Request: Add GUI to Add/Remove Panels

[Aaron Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://mattr.info/r/290/#review295
-----------------------------------------------------------

Ship it!


we will need to spiffy up the Add Panel action (e.g. make it not-PanelContainment specific), but this is a good start for it.



trunk/KDE/kdebase/workspace/plasma/containments/panel/panel.cpp
<http://mattr.info/r/290/#comment214>

    perhaps "Remove" instead of "Delete"? this is in line with the text used in the action, as well.


- Aaron


On 2008-03-12 15:53:17, Stephan Binner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://mattr.info/r/290/
> -----------------------------------------------------------
> 
> (Updated 2008-03-12 15:53:17)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Add a context menu entry to desktop to add a new panel and one to panels to destroy the panel. Requires http://mattr.info/r/286/ to work properly.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/plasma/containments/desktop/desktop.h
>   trunk/KDE/kdebase/workspace/plasma/containments/desktop/desktop.cpp
>   trunk/KDE/kdebase/workspace/plasma/containments/panel/panel.h
>   trunk/KDE/kdebase/workspace/plasma/containments/panel/panel.cpp
> 
> Diff: http://mattr.info/r/290/diff
> 
> 
> Testing
> -------
> 
> Added and removed panels like crazy... ;-)
> 
> 
> Thanks,
> 
> Stephan
> 
>