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

Fix pausing issues when using Web Audio samples #93362

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

adamscott
Copy link
Member

@adamscott adamscott commented Jun 19, 2024

Fixes an issue identified by @Maran23 when pausing the game when using sample playback on the Web platform (#91382).

Sorry for the delay. Tested right now with then newest code, I got another error now when unpausing the game (and therefore all audio as well): image image
#91382 (comment)

Now, the samples should pause and unpause without any issue.

@Maran23

This comment was marked as resolved.

@adamscott adamscott force-pushed the fix-web-audio-pause branch from af063bd to ca77fb9 Compare June 19, 2024 21:48
@adamscott

This comment was marked as resolved.

@Maran23

This comment was marked as resolved.

@Maran23

This comment was marked as resolved.

@adamscott adamscott force-pushed the fix-web-audio-pause branch from ca77fb9 to bc9fb27 Compare June 20, 2024 12:47
@adamscott

This comment was marked as resolved.

@adamscott

This comment was marked as resolved.

@Maran23

This comment was marked as resolved.

@adamscott adamscott force-pushed the fix-web-audio-pause branch from bc9fb27 to 9559dc5 Compare June 20, 2024 13:20
@adamscott

This comment was marked as resolved.

@Maran23

This comment was marked as resolved.

@adamscott adamscott force-pushed the fix-web-audio-pause branch from 9559dc5 to 78c1cac Compare June 20, 2024 14:28
@adamscott

This comment was marked as resolved.

@adamscott adamscott force-pushed the fix-web-audio-pause branch from 78c1cac to 247ede3 Compare June 20, 2024 14:31
@adamscott adamscott force-pushed the fix-web-audio-pause branch from 247ede3 to 57db018 Compare June 20, 2024 14:32
@Maran23
Copy link
Contributor

Maran23 commented Jun 20, 2024

Can confirm, everything looks (and sounds) good now on my end!

Copy link
Contributor

@Maran23 Maran23 left a comment

Choose a reason for hiding this comment

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

As written before, looks good and fixes the issues. Code looks good too. Thanks!

@akien-mga akien-mga merged commit ee3b31d into godotengine:master Jun 21, 2024
17 checks passed
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants