[Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.

[Anurag Chowdhury]

Then I may have misinterpreted Gary's comment.I apologize in that case.

But when I tested Martin's patch on my XO-1.5 , 0.88.1 build  , upon
starting an activity the pulsing icon animation was replaced by a
static grayscale svg icon of the activity ,whose screenshots I have
placed at http://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds.

To reach to the above results I applied the changes suggested at
http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch

Please let me know if I needed to add any more changes to get the
patch working because the sole application of this patch doesn't
produce the said changes at my end.

Also I would wish to ask for Gary's experience after he applied my
patch which I think he once referred to in the previous issue thread.
I worked on the pointers provided there by Gary and would love further
suggestions.

On Fri, Nov 5, 2010 at 10:09 PM, Gary Martin <garycmartin at googlemail.com> wrote:
> Hi Anurag,
>
> On 5 Nov 2010, at 16:19, Anurag Chowdhury <anurag at seeta.in> wrote:
>
>> On Fri, Nov 5, 2010 at 6:53 PM, Martin Dengler <martin at 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
>> Dextrose at lists.sugarlabs.org
>> http://lists.sugarlabs.org/listinfo/dextrose
>