-
Notifications
You must be signed in to change notification settings - Fork 3
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
UI sounds were enabled unintentionally. #177
Comments
My recommendation for addressing this in Equality Explorer 1.1 is described in phetsims/vegas#77 (comment). |
11/18/21 design meeting: The workaround in phetsims/vegas#77 (comment) was approved. I'll need to:
|
Fixes for phetsims/vegas#98 were cherry-picked in phetsims/vegas@e2629f8 and phetsims/scenery-phet@8a4f7f6. |
Work is done in master and 1.1 branches. @KatieWoe would you please assign someone from QA to test this? If this looks OK, change the label to "fixed-awaiting-deploy". To test:
|
Looks good on master |
To verify in 1.1.0-rc.2, follow the steps in #177 (comment). |
Looks good in RC.2 |
For phetsims/qa#735.
While testing this sim for the 1.1 release, I suddenly realized that UI sounds have been enabled. We didn't do this intentionally, and this sim has never had a sound design, or even a discussion about sound.
@jbphet enabled all UI sounds when he switched to tambo-based for
GameAudioPlayer
, common code (in vegas) which plays sounds related to the game. For Equality Explorer, this was first done in the main.js (22f9aad), then moved to package.json (8bf530b). And according to phetsims/vegas#77 and phetsims/chipper#806, this was done for ALL sims that useGameAudioPlayer
.@amanda-phet @kathy-phet
The text was updated successfully, but these errors were encountered: