Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add lazy-loading for <video poster> #8428

Closed
wants to merge 2 commits into from
Closed

Add lazy-loading for <video poster> #8428

wants to merge 2 commits into from

Conversation

zcorpan
Copy link
Member

@zcorpan zcorpan commented Oct 25, 2022

Add a new posterloading attribute to the video element which controls lazy-loading of the poster image.

Fixes #6636.

(See WHATWG Working Mode: Changes for more details.)


/indices.html ( diff )
/media.html ( diff )
/urls-and-fetching.html ( diff )

Add a new posterloading attribute to the video element which controls lazy-loading of the poster image.

Fixes #6636.
@zcorpan
Copy link
Member Author

zcorpan commented Nov 10, 2022

Merge conflict fixed.

Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This largely looks correct to me, modulo some formatting concerns.

source Show resolved Hide resolved
source Show resolved Hide resolved
source Show resolved Hide resolved
@zcorpan
Copy link
Member Author

zcorpan commented Nov 11, 2022

@annevk thanks, fixed.

@zcorpan zcorpan added the do not merge yet Pull request must not be merged per rationale in comment label Mar 20, 2023
@zcorpan
Copy link
Member Author

zcorpan commented Mar 20, 2023

Adding do not merge yet until we align on the approach (new attribute vs use an img element), see #6636 (comment)

@zcorpan
Copy link
Member Author

zcorpan commented Jan 27, 2025

Closing based on recent discussion (#6636 (comment) onwards).

@zcorpan zcorpan closed this Jan 27, 2025
@annevk annevk deleted the posterloading branch January 27, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addition/proposal New features or enhancements do not merge yet Pull request must not be merged per rationale in comment impacts documentation Used by documentation communities, such as MDN, to track changes that impact documentation topic: media
Development

Successfully merging this pull request may close these issues.

Lazy loading video element poster attribute
2 participants