-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add ability to set the sonification level #12
Comments
We discussed this in the 6/14/2018 developer meeting and decided to put this on the PhET menu, and have it behave the way the "Full Screen" option behaves, which is to say that selecting it will turn on enhanced sound and will put a check mark in the menu next to the item. There was some concern that having this control be separated from the sound on/off button wasn't ideal, but @emily-phet pointed out that this control will eventually be part of a larger set of accessibility-related controls that will be grouped together and available via the PhET menu, so having it on the menu (albeit by itself for now) is a reasonable choice. |
@emily-phet - how about having the label for this be "Enhanced Sound"? |
…ambo#12, phetsims/joist#497 Signed-off-by: Chris Malley <[email protected]>
…rder, phetsims/tambo#12, phetsims/joist#497 Signed-off-by: Chris Malley <[email protected]>
I've used the "Enhanced Sound" string, and it's been in place for a while with no objections. I'm going to go ahead and close this issue. |
Sonification supports two levels, "basic" and "enhanced". The user needs to be able to select the setting.
The text was updated successfully, but these errors were encountered: