Introduce mediatypes directory with image type data#27168
Conversation
|
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs). |
5ccaaec to
e1e70dd
Compare
|
Something I think is essential to landing this would be a documented (and very sharp) definition of what qualifies as support for these features. Though they implicate support for HTML, CSS, and web API features, the new top-level namespace implies a different scope. It'd be great to be explicit about what that scope is. (Automated tests could also serve as a definition of supported, but I don't know how practical that is.) |
|
We discussed this in today's BCD meeting, and agreed to (1) compare the data with caniuse to spot any important omissions, and to also (2) ask @Fyrd if and how he would merge this data into the existing caniuse data for image formats. |
caugner
left a comment
There was a problem hiding this comment.
Compared the data with Can I use.
media/formats/image.json
Outdated
| "version_added": "23" | ||
| }, | ||
| { | ||
| "version_added": "17", |
There was a problem hiding this comment.
According to https://caniuse.com/webp, Chrome 9 added WebP support, but I couldn't confirm this with my own tests. It was possibly only behind a flag.
|
What's being proposed here for image formats looks sensible to me. Even though images can be used in many contexts ( For audio and video codecs, it's a bit trickier since it's a combination of container and multiple codecs. https://caniuse.com/webm tracks a specific combination of container and codecs, but https://caniuse.com/av1 tracks a specific video codec without reference to a container. There's also a more complicated history of support for codecs in specific containers, having decode-only vs. decode and encode, etc. But this PR isn't about audio and video codecs, so we don't need to settle those questions here. |
media-types directory with image type datamediatypes directory with image type data
|
|
||
| - `elements` - Elements | ||
|
|
||
| ### [`mediatypes`](./mediatypes) |
0082507 to
d2e33a7
Compare
There was a problem hiding this comment.
LGTM.
Before merging I would like to discuss in tomorrow's BCD call about how we're feeling about this now and if it presents a minor(?) version bump. Last time we introduced a new top-level folder, it was in a breaking BCD 6.0.0 release (#26291).
Co-authored-by: Daniel D. Beck <daniel@ddbeck.com>
Summary
Introduces the
mediatypestop level directory, and adds image types undermediatypes.image.Test results and supporting details
Sources:
Related issues
Part of #6971.