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