Hi Anurag,

On 5 Nov 2010, at 16:19, Anurag Chowdhury <anu...@seeta.in> wrote:

> On Fri, Nov 5, 2010 at 6:53 PM, Martin Dengler <mar...@martindengler.com> 
> wrote:
>> On Fri, Nov 05, 2010 at 06:05:16PM +0530, Anurag Chowdhury wrote:
>>> Sure , we can work to get a combined patch but your patch still
>>> disables the animation in most of the icons
>> 
>> I think you're talking about something else than my patch.  No
>> animation is disabled.  Not sure what's wrong but the patch you linked
>> to[1] in your wiki page clearly does nothing like disabling any
>> animation.
> Well, I actually didn't meant that. I was talking about the change
> noticed by Gary

Hmmm, no, with Martin's below linked patch pulse colour animation is good.

The only extra thing I noted in my email was that the 1sec zoom effect is still 
not showing for anything by very simple icons (Log icon is simple enough, 
Distance is not), however the zoom effect is not showing with your 'skip the 
first update' approach either. Fixing zoom efficiency should be looked at as a 
separate issue.

--Gary 

> (and me too) i.e. after the application of the patch
> we don't get to see the pulsing icon animation in most of the icons ,
> all we see is only the grayscale svg icon , this anomaly is also
> linked with the complexity of the icons i.e. more complex icons don't
> show up the animation while the simpler ones do.
>> 
>>> Regards
>>> Anurag
>> 
>> Martin
>> 
>> 1. 
>> http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch
>> 
> _______________________________________________
> Dextrose mailing list
> dextr...@lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/dextrose
_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to