[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: [PATCH] Animations enable/disable system wide
From: "Aaron J. Seigo" <aseigo () kde ! org>
Date: 2008-02-19 5:18:33
Message-ID: 200802182218.33851.aseigo () kde ! org
[Download RAW message or body]
On Friday 15 February 2008, Rafael Fernández López wrote:
> Yes, I have thought on that too... for now we can let the accessors and
> think la ter on how can we modify this on runtime depending on the
> remaining enery (or go back to the normal settings if we for instance, were
> plugged on).
the how is pretty easy: Solid::PowerManagement. the when is a bit more
involved, particularly figuring out what to put in the user interface for the
user to work with.
> > Rafael's patch is a step in the right direction at least ... and now it's
> > easy to find what needs to get fixed if we modify this API via an lxr
> > search ;)
>
> Before getting this into the public API, I think we should think well about
> this , now changes should be thought twice before committing, specially in
> topics lik e this one. I vote for changing the bool for an int, so we don't
> shoot ourselves on the feet.
well, an enumeration in any case. =)
> How to call those methods, and what values the enum should have ?
graphicEffectsLevel()?
glamour()? ;) we could have Jenine Gorafalo at the low end and Angelina Jolie
at the high end? (yeah, yeah, i'm joking)
i'd make it a set of flags (so multiple values can be |'d together), probably
somethin glike:
GraphicEffect {
NoEffects = 0,
GradientEffects = 1,
SimpleAnimationEffects = 2,
ComplexAnimationEffects = 4 };
Q_DECLARE_FLAGS(GraphicEffects, GraphicEffect)
i don't know what other flags might be desired/wanted. i could ask Zack when
he gets here in a few days if he has any ideas...
--
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 Trolltech
["signature.asc" (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic