'Next wallpaper' functionality for image plugin?

[Chani]

On September 26, 2009 11:03:37 Aaron J. Seigo wrote:
> On September 26, 2009, Yuen Hoe Lim wrote:
> > the wallpaper plugin doesn't have control over the right-click-desktop
> > context menu, and it will be inappropriate to hack the functionality
> > into the core desktop code.
> 
> that's correct; but we already have a way to add context menu items from
> applets and what not. the containment is in complete control here, and the
> containment is also what knows about the wallpaper. so it could ask the
> wallpaper if it has anything to add actions to add to the context menu.
>  should be easy to add some mechanism to the Wallpaper plugin API for this.
> 

heck yeah. add a contextualActions function and have the default 
implementation of Containment::contextualActions call it if wallpaper exists.

con: custom desktop containments might break that unintentionally
pro: custom desktop containments can intentionally stop it if they want to.

if we don't think the containment plugin should be allowed to block wallpaper 
actions then just add a separate function in Containment, 
wallpaperContextualActions, and have the contextmenu plugin draw from that 
too. or if ContainmentActions plugins can access the containment's wallpaper 
object directly we don't need a function to forward the data. :)


-- 
This message brought to you by eevil bananas and the number 3.
www.chani3.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20090928/5c2ce645/attachment.sig