Sugar Labs

Pulsing icon delayed by 5 seconds: Difference between revisions

← Older edit
No edit summary
add gonzalo's solution
 
(8 intermediate revisions by 2 users not shown)
Line 1: Line 1:
==Pulsing icon delayed by 5 seconds==
==Pulsing icon delayed by 5 seconds==


===Description===
===Description===
Whenever an activity is started in sugar OS , we get to see a few seconds delay after clicking on the activity icon on the home screen and before the appearance of the pulsing icon animation. The delay also seems to be directly proportional to the complexity of the icon of the activity.
SugarLabs bug [http://bugs.sugarlabs.org/ticket/2080 #2080] was raised because whenever an activity is started in sugar OS , we get to see a few seconds delay after clicking on the activity icon on the home screen and before the appearance of the pulsing icon animation. The delay also seems to be directly proportional to the complexity of the icon of the activity.


===Solutions and their End Results===
===Proposed solutions and their End Results===




Line 26: Line 27:
'''Result :''' The idea was to reduce the rendering time of the first frame so as to speed up the appearance of the first frame of the animation and hence reduce the delay. This fix seems to work as per expectations and has reduced the delay time by more than  50% as noted in the logs and reported by the community.We can see the below attached screenshots showing the visual effects,  after the application of the patch on an XO-1.5  
'''Result :''' The idea was to reduce the rendering time of the first frame so as to speed up the appearance of the first frame of the animation and hence reduce the delay. This fix seems to work as per expectations and has reduced the delay time by more than  50% as noted in the logs and reported by the community.We can see the below attached screenshots showing the visual effects,  after the application of the patch on an XO-1.5  
<br>
<br>
[[File:Patch_v5_result_1.jpg|600px]]<br>
[[File:Patch_v5_result_1.jpg|200px]]
'''Screenshot at t =t0 '''<br>
[[File:Patch_v5_result_2.jpg|200px]]
[[File:Patch_v5_result_2.jpg|600px]]<br>
[[File:Patch_v5_result_3.jpg|200px]]
'''Screenshot at t =t1 '''<br>
[[File:Patch_v5_result_4.jpg|200px]]
[[File:Patch_v5_result_3.jpg|600px]]<br>
 
'''Screenshot at t =t2 '''<br>
[[File:Patch_v5_result_4.jpg|600px]]<br>
'''Screenshot at t =t3 '''
<br>
<br>
''t3 > t2 > t1 > t0''
''The screenshots sequentially show the frames of the pulsing icon animation after the application of the above fix''
<br>.
<br>.
===FIX # 5 ===
===FIX # 5 ===
Here the idea was suggested and implemented by Martin dengler <martin@martindengler.com>. We can find his patch  [http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch here]. <br>
Here the idea was suggested and implemented by Martin dengler <martin@martindengler.com>. We can find his patch  [http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch here]. <br>
Line 42: Line 41:
'''Result :''' The applied patch reduces the delay by more then 60%, as confirmed by the community, but it's worth a note that the zoom in effect is still lost for most svg icons, only for the very simple icons effect is visible (Log icon is simple enough to show it, Distance icon is not quick enough).We can see the below attached screenshots showing the visual effects,  after the application of the patch on an XO-1.5  
'''Result :''' The applied patch reduces the delay by more then 60%, as confirmed by the community, but it's worth a note that the zoom in effect is still lost for most svg icons, only for the very simple icons effect is visible (Log icon is simple enough to show it, Distance icon is not quick enough).We can see the below attached screenshots showing the visual effects,  after the application of the patch on an XO-1.5  
<br>
<br>
[[File:Patch_MD_result_1.jpg|600px]]<br>
[[File:Patch_MD_result_1.jpg|200px]]
'''Screenshot at t =t0 '''<br>
[[File:Patch_MD_result_2.jpg|200px]]
[[File:Patch_MD_result_2.jpg|600px]]<br>
[[File:Patch_MD_result_3.jpg|200px]]
'''Screenshot at t =t1 '''<br>
[[File:Patch_MD_result_4.jpg|200px]]
[[File:Patch_MD_result_3.jpg|600px]]<br>
'''Screenshot at t =t2 '''<br>
[[File:Patch_MD_result_4.jpg|600px]]<br>
'''Screenshot at t =t3 '''
<br>
<br>
''t3 > t2 > t1 > t0''
''The screenshots sequentially show the frames of the pulsing icon animation after the application of the above fix.''
<br>.
<br>.
===FIX # ===
Gonzalo came up with a patch that uses Cairo to eliminate much of the re-drawing.  He published [http://dev.laptop.org/~gonzalo/alpha-pulsingicon/ a version] that is now being polished for merging.
==Conclusion==
Gonzalo's fix is preferred.


===Contact===
===Contact===
Retrieved from "https://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds"