[PATCH] Review request for plasma marble wallpaper (playground)

[Aaron J. Seigo]

On Monday 15 December 2008, Sascha Peilicke wrote:
> Mouse interaction was removed completely, this caused all sort of weird
> troubles when interacting with plasma widgets (instead rotates the globe or
> vice versa). 

are you saying that mouse events passed to the wallpaper would end up passed 
to the widget as well? were you accept()ing the mouse event passed in? if 
these are bugs in libplasma, i'd like to know about them so they can be fixed. 
=)

> Mouse scrolling previously used for zooming the globe is also
> otherwise reserved for switching workspaces.

feel free to override that. you shouldn't worry about such things, really. 
implement your plugin like it's the only thing in the world. er.. marble. ;)

> Maybe we should also change the name from "The Earth" to something more
> descriptive or just "Marble globe" to advertise this great technology ^^

"The Earth" is pretty descriptive, i think =) it's more obvious what it is 
compared to "Marble Globe" as well (i wouldn't know what that was if i didn't 
know that Marble was a KDE mapping application). in plasma we have a policy of 
not showing the technical name of something (jargon) when there is a 
functional name to it. ergo things like Activity instead of Containment.

as for the patch itself, i obviously leave that up to you and Frederic

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20081215/e4224615/attachment.sig