-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
[Media] Enable AVIF support #1726
Comments
Most of this got done when we thought it would go into FF83 - see mdn/sprints#3804 (comment) Actions
|
Yes please feel free to carry on @hamishwillee :) I was only going through and assigning anything relevant to myself. Let me know anyhoo - happy with whatever |
Thanks @Rumyra - I'll take this one then. FYI I've edited my original post to you to become the "status". |
@chrisdavidmills This should be more or less ready for review. I added comment for the dev team just then, so there might be further comments of course. |
DDC set, all work reviewed, outstanding PRs merged. This is an outstanding set of work; thanks @hamishwillee ! |
@hamishwillee Hi Hamish, sorry to land this on you at the 11th hour, but AVIF support has been pulled from 86: https://bugzilla.mozilla.org/show_bug.cgi?id=1682995#c31 As a priority in your next working day, can you revert the changes in this bug as much makes sense? I.e. I don't think you need to revert everything, just explicit mentions of Fx version support in 86. I'll remove it from the 86 rel notes and release blog post. |
OK, removed from rel notes: f4b1336 |
Yoiks. OK
|
Yup, agreed. All reviewed and merged. Thanks! |
Excellent. Closing this again then. Unless we're getting avif back in FF87 of course :-) |
I'd actually like to keep it open; we will definitely be getting it back at some point. |
Closing again. This was actually addressed back in FF92 :-). |
Acceptance criteria
Features to document
So a few places to update information.
Related Gecko bugs
For folks helping with Firefox-related documentation features — make sure above AC have been done, but also:
The text was updated successfully, but these errors were encountered: