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

[Sascha Peilicke]

On Wednesday 17 December 2008 12:53:30 Frederik Gladhorn wrote:
> Hi Sascha,
> thanks for looking at the marble wallpaper - that's much appreciated from
> my side :)
> I have had no time to look at the patch at all yet, will do so on the
> weekend.
I attached a more current one here :-)

>
> On Monday 15 December 2008 21:41:16 Sascha Peilicke wrote:
> > For Frederic Gladhorn and FYI.
> >
> >
> > This patch adds/changes a lot of features to the marble plasma wallpaper
> > in playground.
> >
> > Re-enabled the stars plugin as current Marble trunk doesn't seem to crash
> > on it any further.
>
> Seems good to me, I enabled it again here and have had no trouble with it
> lately.
>
> > Moved to a more generic support for globe rotation schemes to allow
> > something like "hover on home location" or free rotation (with the marble
> > navigation plugin).
> >
> > Mouse interaction was removed completely, this caused all sort of weird
> > troubles when interacting with plasma widgets (instead rotates the globe
> > or vice versa). Mouse scrolling previously used for zooming the globe is
> > also otherwise reserved for switching workspaces. Instead the user can
> > enable the marble navigation widget to zoom/rotate the globe.
>
> I had not a single weird interaction - you just need to differentiate
> between applet and wallpaper and you'll be fine.
> This is the change that I don't agree with. For me the wallpaper is all
> about being fancy-interactive-hotness and it's sole purpose is to brag
> about KDE hotness with it.
> Many people hate the wheel on workspace option, I don't have a strong
> opinion either way, but I think the wheel is much better used to zoom here.
> This wallpaper doesn't bring speed or a faster work flow - it is just a
> toy, so people using the wheel to switch desktops won't use this wallpaper
> I guess. My problem with the marble navigation widgets is that they clutter
> the desktop. I want to see that globe thingy, not lots of controls.
> Maybe a checkbox to enable mouse interaction? More options ftw...?
Yeap, this seems to need a bit more discussion, here's what I currently have:

4 rotation modes, 2 already known (follow sun, custom rot), + hover on home + 
free movement. free movement basically means no movement + a checkbox which 
enables the marble navigation widget. I agree that it would be a bit clumsy to 
be shown all the time, so this is more intented to be enabled, move the globe 
in place, disable it and enjoy. A bonus is that the nav widget can be 
customized, so I stripped it down a bit, this is what it looks like when 
enabled:

http://img150.imageshack.us/img150/4710/marblenavwidgetwc3.png

> I started it to show a friend who had something similar once upon a time in
> some proprietary environment how flexible and great our frameworks are.
> I'd even like to open a browser with the placemarks - same as marble does.
> Wikipedia ftw.
Would be great to have, this would be the real semantic desktop ^^

> > Reworked the configuration screen to show only options which make sense
> > together. This means some values are shown and hidden based on selected
> > globe rotation behavior, marble theme etc. More generic "show placemarks"
> > instead of "show night, show citylights, show places" to not present too
> > much config detail to the user.
>
> Very good, I just threw in every marble option I could find ;) making it
> more understandable is a good idea.
>
> > Maybe we should also change the name from "The Earth" to something more
> > descriptive or just "Marble globe" to advertise this great technology ^^
>
> See Aarons reply, I'm fine with Globe, I'm a little against putting Marble
> here.
Updated the desktop file accordingly.


-- 
Sascha Peilicke
http://saschpe.wordpress.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: marble-wallpaper-mod.patch
Type: text/x-patch
Size: 40234 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20081218/8e7d7d09/attachment-0001.patch 
-------------- 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/20081218/8e7d7d09/attachment-0001.sig