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

First sound is quiet #283

Closed
catherinecarter opened this issue Apr 7, 2024 · 3 comments
Closed

First sound is quiet #283

catherinecarter opened this issue Apr 7, 2024 · 3 comments
Assignees

Comments

@catherinecarter
Copy link

In playing with the sim, I noticed in dev.23 that the very first sound in the sim is very quiet, regardless of what I pressed.

Subsequent sounds (including the one that had originally been quiet) were normal volume.

I tested in both Safari and Chrome.

@catherinecarter catherinecarter added type:bug Something isn't working and removed type:bug Something isn't working labels Apr 7, 2024
@catherinecarter
Copy link
Author

@samreid noted that this is a tambo thing, where the browser needs the user to click anywhere to 'activate' the sound. This is not specific to PDL.
@jbphet - is this something that we'll live with? Or is this worth brainstorming ways to fix?

@catherinecarter catherinecarter assigned jbphet and unassigned samreid Apr 8, 2024
@jbphet
Copy link
Contributor

jbphet commented Apr 8, 2024

It is not a "tambo thing", it is a "browser thing". Browsers are prevented from playing sounds until the user interacts with the page in some way. There is a lot of information on the cause and our handling of it in phetsims/tambo#147 and phetsims/tambo#9.

Note that in the first issue I linked, it says "We've basically just decided to live with this for now, so I'll mark the issue as deferred. However, if we ever decided it was a big enough problem that we should address it, we could add some code to make the initial volume changes really quick, and to also delay the first playing of a sound until these were all done." So, if you think this rises to the level where we should implement better handling of it, be sure to chime in on the issue and raise it for some possible attention in upcoming iterations.

@jbphet jbphet assigned catherinecarter and unassigned jbphet Apr 8, 2024
@matthew-blackman
Copy link
Contributor

Since this is a known issue and is being tracked elsewhere, this issue can be closed.

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

No branches or pull requests

4 participants