The Video page needs to lazy load. about ourumbraco HOT 4 CLOSED

@readingdancer Good catch 👍

I think my original ideas was to have some kind of pagination as the list would grow in size. So I think that would be the ideal solution, but also requires some extra work.

Lazy loading should however be somewhat simpler - I've played around with some JavaScript, which resulted in this JSFiddle:

https://jsfiddle.net/uem4kLfs/show

Once you scroll, it detects the images (with the lazyload class) that enters the viewport. I will look into implementering that in both video lists (community and HQ).

from ourumbraco.

@abjerner Yes, this is exactly what I was going to implement, great you have managed to do it already :)

from ourumbraco.

Hi @readingdancer,

We're writing to let you know that we've added the Up For Grabs label to your issue. We feel that this issue is ideal to flag for a community member to work on it. Once flagged here, folk looking for issues to work on will know to look at yours. Of course, please feel free work on this yourself ;-). If there are any changes to this status, we'll be sure to let you know.

For more information about issues and states, have a look at this blog post

Thanks muchly, from your friendly PR team bot :-)

from ourumbraco.

This issue has been in the up for grabs state for quite a while now and it seems nobody is ready to pick this one up.

For now we'll close this issue to prevent the list of up for grabs from becoming very stale. We're happy to re-open it if someone still thinks it's good to work on this.

If anyone is about to pick this issue up to fix it, make sure to test first if you can reproduce the problem in the latest version before you start to work on it.

Thanks!
Sebastiaan on behalf of Umbraco HQ.

from ourumbraco.