<div class="gmail_quote"><div>The present issue has two possible scenarios of solving it</div><div><br></div><div>1) Optimise the present pulsing icon animation to reduce the delay : Thats what I have worked towards till now in my patches.</div>
<div><br></div><div>2)Replace the present pulsing icon animation with a better and faster animation: Thats what I think everyone agrees to be the best solution of the issue right now.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">
</div>If we're still taking about the v5 patch that was posted to the list, I<br>
don't understand how this change is supposed to fix the bug:<br>
<div class="im"><br>
 +        if self._count > 2:<br>
 +           self.update()<br>
 +        self._count = self._count + 1<br>
<br></div></blockquote><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; "><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
<div>The animation of the pulsing icon is basically constituted by filling the raw svg icon with colours based on a sinusoidal function</div></blockquote><div>    and it is this sinusoidal filling of the colour that brings about the pulsing effect , so if we dont fill the icon with colour then </div>
<div>    the icon wont pulse and would render and appear as a raw svg icon only which undoubtedly will take lower system resources </div></span><div><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; ">    to get rendered than the animation.</span> </div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">
</div>Skipping the first frame of the animation unconditionally is wrong and<br>
isn't the same thing of skipping frames dynamically, based on the time<br>
elapsed to render  the previous frame. I feel like we're trying a number<br>
of random tweaks without addressing the root cause of the problem.<br></blockquote><div><br></div><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; "><div> The first frame of the animation is not being skipped but its just not being filled with colour so as to make the pulsing icon animation start earlier and I hope we both agree on the fact that the delay is caused due to the time taken for the rendering of the zoom in and zoom out animation for the first  frame. So if the load of processing the first frame would have been reduced somehow then the animation would start earlier and this approach was confirmed by positive log results upon benchmarking the whole scenario before and after the fix. </div>
</span><div><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; "></span> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
Perhaps Anurag could work on a fix in team with other Seeta developers?<br>
<div><div></div><div class="h5"><br>
--<br>
   // Bernie Innocenti - <a href="http://codewiz.org/" target="_blank">http://codewiz.org/</a><br>
 \X/  Sugar Labs       - <a href="http://sugarlabs.org/" target="_blank">http://sugarlabs.org/</a><br>
<br>
<br>
</div></div></blockquote></div><br>