Skip to content

Record Safari 18.4+ support for Opus in Ogg containers #39592

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

acdha
Copy link

@acdha acdha commented May 20, 2025

Description

Confirm support for Opus and Vorbis in Ogg containers on Safari 18.4+

Motivation

People providing audio content on the web want to know what formats they must support.

Additional details

@acdha acdha requested a review from a team as a code owner May 20, 2025 16:30
@acdha acdha requested review from dipikabh and removed request for a team May 20, 2025 16:30
@github-actions github-actions bot added Content:Media Media docs size/s [PR only] 6-50 LoC changed labels May 20, 2025
Copy link
Contributor

github-actions bot commented May 21, 2025

Preview URLs

(comment last updated: 2025-05-29 14:02:29)

Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

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

Hi @acdha, thanks for this update. The added info looks good.

I have a couple of suggestions:

  • Since we don't include versions for other browsers in the table, could we move the Safari 18.4+ info out of the table and add it as a short note after the table? We can include Vorbis support in the same note. To give you an idea of the markdown we use to add a note callout:
> [!NOTE]
> Safari 18.4+ (on macOS 15.4+, iOS 18.4+, iPadOS 18.4+, and visionOS 2.4+) added support for Opus and Vorbis codecs in Ogg containers.

Ideally, version-specific info should go in the table in the Browser compatibility section. But I don't see that on this page.

In any case, do you want to add "Safari" to the "Ogg audio/ogg" row in the browser compat table as well?

@acdha
Copy link
Author

acdha commented May 28, 2025

In any case, do you want to add "Safari" to the "Ogg audio/ogg" row in the browser compat table as well?

That makes sense to me logically, but the structure of the final table is a bit odd since the “Browser support” column has only single values for anything. It seems like that could either be a comma-separated list or separate columns like the earlier tables. Do you have a preference?

@dipikabh
Copy link
Contributor

In any case, do you want to add "Safari" to the "Ogg audio/ogg" row in the browser compat table as well?

That makes sense to me logically, but the structure of the final table is a bit odd since the “Browser support” column has only single values for anything. It seems like that could either be a comma-separated list or separate columns like the earlier tables. Do you have a preference?

Comma should be good for this one. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Media Media docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants