Skip to content

Use html audio element to call setSinkId for reactions and call sounds. #3321

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

Draft
wants to merge 1 commit into
base: livekit
Choose a base branch
from

Conversation

toger5
Copy link
Contributor

@toger5 toger5 commented Jun 5, 2025

setSinkId on the audio context is not supported on safari ios. So we try using an html audio element and audioElement.setSinkId instead with this PR.

@toger5 toger5 requested a review from a team as a code owner June 5, 2025 09:53
@toger5 toger5 requested a review from AndrewFerr June 5, 2025 09:53
@toger5 toger5 added the PR-Improvement Release note category. A PR that improves EC's performance or stability. label Jun 5, 2025
@toger5 toger5 force-pushed the toger5/html-audio-for-reactions branch from ab9298c to 48a1240 Compare June 5, 2025 12:39
@toger5 toger5 changed the title Use html audio element to call setsink id for reactions and call sounds. Use html audio element to call setSinkId for reactions and call sounds. Jun 5, 2025
@toger5 toger5 force-pushed the toger5/html-audio-for-reactions branch from 576abb2 to aecb550 Compare June 5, 2025 16:08
@toger5
Copy link
Contributor Author

toger5 commented Jun 12, 2025

With more testing on the subject of audio devices I suspect there is hope that this is not needed. lets keep this in draft until I know for sure.

@toger5 toger5 marked this pull request as draft June 12, 2025 14:33
@toger5 toger5 marked this pull request as draft June 12, 2025 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Improvement Release note category. A PR that improves EC's performance or stability.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant