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

web: On browsers which support it, specify the type of audio Ruffle uses #18752

Merged
merged 3 commits into from
Dec 1, 2024

Conversation

danielhjacobs
Copy link
Contributor

I've not tested this, but it may fix ruffle-rs/ruffle-rs.github.io#177.

@danielhjacobs danielhjacobs added A-web Area: Web & Extensions T-fix Type: Bug fix (in something that's supposed to work already) labels Nov 25, 2024
@n0samu
Copy link
Member

n0samu commented Nov 26, 2024

Looks good, but wouldn't it make more sense to do this in the unmuteAudioContext() function?

private unmuteAudioContext(): void {

See also: #1552

@danielhjacobs
Copy link
Contributor Author

Done, but I won't feel right merging this until someone with iOS 17+ tests it.

@danielhjacobs
Copy link
Contributor Author

danielhjacobs commented Dec 1, 2024

Can someone with iOS 16.4+ test https://danielhjacobs.github.io/ruffle-unmute-ios/ with the mute switch enabled and make sure it still plays audio.

@danielhjacobs
Copy link
Contributor Author

Tested and confirmed fixed by @CUB3D on Discord.

Copy link
Contributor

@CUB3D CUB3D left a comment

Choose a reason for hiding this comment

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

Tested and confirmed working on iOS 16.6.1

@danielhjacobs danielhjacobs enabled auto-merge (rebase) December 1, 2024 19:03
@torokati44
Copy link
Member

Also by a friend of mine on an iPhone with iOS 18.

@danielhjacobs danielhjacobs merged commit 2eb5ff3 into ruffle-rs:master Dec 1, 2024
20 checks passed
@danielhjacobs danielhjacobs deleted the specify-audio-session branch December 1, 2024 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-web Area: Web & Extensions T-fix Type: Bug fix (in something that's supposed to work already)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Audio not working on Safari iOS/iPad OS when mute is switched on
4 participants