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

Dan Meltzer parallelgrapefruit at gmail.com
Mon Feb 4 19:43:59 CET 2008



> On 2008-02-04 12:31:07, Jeremy Whiting wrote:
> > Looks good to me

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?


- Dan


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


On 2008-02-04 12:12:52, Aaron Seigo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://matt.rogers.name/r/69/
> -----------------------------------------------------------
> 
> (Updated 2008-02-04 12:12:52)
> 
> 
> 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