Hi Chani,<br><br>Thanks for the help! That helped to clear up a lot :)<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
then, there are two ways you could do this:<br>
1) create QList<QAction*> Containment::contextualActions() and have it return<br>
wallpaper()->contextualActions() by default and edit every desktop-containment<br>
plasmoid (desktop and folderview at least) to add wallpaper()-<br>
>contextualActions() to its own contextualActions (assuming the function<br>
already exists there).<br>
<br>
2) edit the contextmenu mouse plugin to add containment()->wallpaper()-<br>
>contextualActions() to its actions the same way it adds containment()-<br>
>contextualActions().<br></blockquote><div><br>Think I'll go with option 2. Don't think plasmoids should or need to care about its containment's wallpaper.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
and of course, either way the contextualActions function has to be added to<br>
Wallpaper too... er, except it can't be virtual because that's BIC, damnit. so<br>
you'll have to find a more creative way of letting wallpaper plugins specify<br></blockquote><div><br>I didn't understand that (what's BIC?) but yeah, sure enough plasma crashed on me when I tried to add Wallpaper::contextualActions and make it virtual =( I managed to work around it (satisfactorily I hope), but could you explain what's going on anyway (what's wrong with making it virtual?), so that I can become less puzzled :)<br>
<br>Anyway, I think I have managed to cook up something reasonably presentable. Now to clean up and figure out how to ask for review (yeah, this is my first attempt at a commit outside playground :D )<br><br clear="all">----<br>
Jason "moofang" Lim Yuen Hoe<br><a href="http://yuenhoe.co.cc/">http://yuenhoe.co.cc/</a><br><br></div></div>