mjstraughan / CubingHistory

A history of twisty puzzle contributions.
http://cubinghistory.com
1 stars 1 forks source link

performance issues #12

Open bcube2 opened 6 hours ago

bcube2 commented 6 hours ago

I know it´s not a priority for you right now and I don´t want to overwhelm you, but later you might want to address performance issues. For example, https://www.cubinghistory.com/World%20Records/WorldRecordSingles2H takes a very long time to load, causing that the first simulator is laggy in animation on my mobile phone.

Performance summary can be seen e.g. in an automatic performace test tool such as PageSpeed Insights.

mjstraughan commented 5 hours ago

I definitely noticed and it has been annoying me. Not yet sure if it is the large number of YouTube embeds, simulators, or the combination. I'll do some testing a little later and see if placing the videos in a collapsible improves the performance. I originally planned to have all or most of the videos in collapsibles anyway.

mjstraughan commented 4 hours ago

Using that recommended tool, it looks like it is the YouTube embeds.

bcube2 commented 4 hours ago

Not yet sure if it is the large number of YouTube embeds, simulators, or the combination.

In PageSpeed Insights, you can test a page with 142 simulators and no performance optimization whatsoever: https://bcube2.github.io/old_waterman_method/stage3.html or a page with less simulators but also some youtube videos of former WRs with some basic performance optimization: https://hlavolam.maweb.eu/rubiks-cube-3x3x3 which should result in a conclusion that a lot of ACJS simulators even without any kind of performance optimization is not really an issue in terms of performance.

mjstraughan commented 4 hours ago

I added loading="lazy" to all YouTube embeds on the records pages. The performance is better for me and the embeds on PageSpeed Insights is showing a loading time of 0 for most. Still not perfect I think. How is it on your side?