[Panel-devel] KDE/kdebase/workspace

[Sebastian Sauer]

Aaron J. Seigo wrote:

> On Monday 30 July 2007, Sebastian Sauer wrote:
>> Would it be an idea to provide access to the KMenu in
>> DesktopView::contextMenuEvent somehow to be able to extend the
>> contextmenu with applet related things? Probably just allow an applet to
>> define an own i18n("%1 ...",applet->name()) submenu or allow to merge
>> menus?
> 
> yes; this has been discussed a couple times and we all agree it's a good
> idea. it's just a matter of finding a nice way to do it (API wise) and
> writing the code that Makes It So. in fact, it's even on the plasma Tasks
> page in the Applets section =)
 
3 proposals;

plasma1.patch goes the most easy way to just allow an applet to change the 
menu. Probably it would be also an idea to just pass a 
KActionCollection/KAction's and add them to the menu cause that way we arn't 
limited to a kmenu for the context-stuff

plasma2.patch just redirects the request to the applet, no KMenu "merging" but 
it looks even shorter ;)

plasma3.patch just combines 1 and 2 :)

Ideas for alternates?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: plasma1.patch
Type: text/x-diff
Size: 1076 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20070731/4fadfa37/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: plasma2.patch
Type: text/x-diff
Size: 503 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20070731/4fadfa37/attachment-0001.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: plasma3.patch
Type: text/x-diff
Size: 1379 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20070731/4fadfa37/attachment-0002.bin