Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Lazy load YouTube embedded video references #57

Open
Kallie-Riff opened this issue Feb 10, 2017 · 2 comments
Open

Lazy load YouTube embedded video references #57

Kallie-Riff opened this issue Feb 10, 2017 · 2 comments
Assignees

Comments

@Kallie-Riff
Copy link
Contributor

Currently the tech-talk tab loads all videos into the page during render.
These videos should be 'lazy loaded' onto the page - so when the user clicks play the video will load but not before that happens. We will use the same design for the upcoming internal speakers tab.

Thinking about using something similar to this to solve the issue.

If load speeds are still slow after this change we could consider looking at pagination (might want this anyway for better usability).

@anelson425
Copy link
Contributor

This is a good strategy, loading the content during render is less than ideal, especially as the content grows. Lazy loading them will allow viewers pick and choose what is worth loading to them.

@cchesser
Copy link
Member

I'm going to pull this into our uplift of the site. We are doing doing an embedded YouTube video inclusion, but can optimize the generated HTML for the embedded view. In this uplift, we are also using the latest of Bootstrap, which also has options:

https://medium.com/@ryanlebel/speed-up-page-load-time-c6b9148d0a2

@cchesser cchesser changed the title Youtube videos - Lazy loading Lazy load YouTube embedded video references Jul 15, 2020
@cchesser cchesser self-assigned this Jul 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants