Review Request: Add Containment contextActions() to widget context menu

Aaron Seigo aseigo at kde.org
Tue Feb 5 05:28:30 CET 2008



> On 2008-02-04 12:31:07, Jeremy Whiting wrote:
> > Looks good to me
> 
> Dan Meltzer wrote:
>     Hmm, does the keep the "Add Widget" entry?  It looks like it will, but I doubt it should.. what would happen if you added a widget to a widget?

updated patch now pushes this all into a submenu when there is more than one entry


- Aaron


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://matt.rogers.name/r/69/#review59
-----------------------------------------------------------


On 2008-02-04 22:28:18, Aaron Seigo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://matt.rogers.name/r/69/
> -----------------------------------------------------------
> 
> (Updated 2008-02-04 22:28:18)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Injects the containment's context menu into the widget's. This allows, e.g. "Configure Panel" to show up in the context menu.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/libs/plasma/containment.cpp
> 
> Diff: http://matt.rogers.name/r/69/diff
> 
> 
> Testing
> -------
> 
> Right clicking on panel, widgets in panel, widgets on desktop, desktop. All worked as expected.
> 
> 
> Thanks,
> 
> Aaron
> 
>



More information about the Panel-devel mailing list