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

Policy/linting for {% Video %} tag embeds? #24

Open
jeffposnick opened this issue Feb 18, 2022 · 2 comments
Open

Policy/linting for {% Video %} tag embeds? #24

jeffposnick opened this issue Feb 18, 2022 · 2 comments
Labels
question Further information is requested

Comments

@jeffposnick
Copy link
Contributor

The {% Video %} embed being added to a docs page at GoogleChrome/developer.chrome.com#2143 seems like it could lead to a bad experience for users:

  • We allow an embedder to set autoplay regardless of the size of the underlying video. The video itself is 17.2mb, which is much smaller than an animated GIF would be, but still a much larger download that you'd expect when you visit a docs page. I believe our media uploader supports files up to 25mb.
  • We don't do anything that I can see to respect prefers-reduced-motion or prefers-reduced-data.

Some of these might be addressable via code changes, and some might be addressable via linting rules (i.e. only allow autoplay when the video is is smaller than some threshold).

@jeffposnick
Copy link
Contributor Author

CC: @rachelandrew @devnook for visibility.

@jecfish
Copy link
Contributor

jecfish commented Feb 22, 2022

+1 to this. Linting rule for video smaller than certain threshold.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants