[PATCH] Animations enable/disable system wide

Aaron J. Seigo aseigo at kde.org
Tue Feb 19 05:18:33 GMT 2008


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080218/8e583642/attachment.sig>


More information about the kde-core-devel mailing list