[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: 'Next wallpaper' functionality for image plugin?
From: Yuen Hoe Lim <yuenhoe86 () gmail ! com>
Date: 2009-10-11 12:58:14
Message-ID: 26362f480910110558q787fba55ya142841da9acb8af () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Hi Chani,
Thanks for the help! That helped to clear up a lot :)
then, there are two ways you could do this:
> 1) create QList<QAction*> Containment::contextualActions() and have it
> return
> wallpaper()->contextualActions() by default and edit every
> desktop-containment
> plasmoid (desktop and folderview at least) to add wallpaper()-
> >contextualActions() to its own contextualActions (assuming the function
> already exists there).
>
> 2) edit the contextmenu mouse plugin to add containment()->wallpaper()-
> >contextualActions() to its actions the same way it adds containment()-
> >contextualActions().
>
Think I'll go with option 2. Don't think plasmoids should or need to care
about its containment's wallpaper.
and of course, either way the contextualActions function has to be added to
> Wallpaper too... er, except it can't be virtual because that's BIC, damnit.
> so
> you'll have to find a more creative way of letting wallpaper plugins
> specify
>
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 :)
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 )
----
Jason "moofang" Lim Yuen Hoe
http://yuenhoe.co.cc/
[Attachment #5 (text/html)]
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>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic