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

Decide whether to turn off UI sounds on next publication #74

Open
markgammon opened this issue Nov 19, 2021 · 2 comments
Open

Decide whether to turn off UI sounds on next publication #74

markgammon opened this issue Nov 19, 2021 · 2 comments

Comments

@markgammon
Copy link

markgammon commented Nov 19, 2021

As part of the work done for phetsims/vegas#77, UI sounds got turned on for this simulation. Before the sim is published again from master, we should decide whether to turn the UI sounds off, leave them on and design and implement sounds for the rest of the interactions, or publish with a partial sound design.

If the decision is to turn them off, this can be accomplished by adding the following line to the end of the sim's -main.js file:

  soundManager.setOutputLevelForCategory( 'user-interface', 0 );
@pixelzoom
Copy link
Contributor

pixelzoom commented Mar 4, 2023

Work on this sim has resumed -- see #77, #78 and #79.

Common code provides good coverage of UI sounds for this sim.

But the spinners (NumberSpinner) need work. There has been no sound design/implementation for NumberSpinner, so it just has the default "button click" sound. To be consistent with other "value" controls (slider, etc.) the pitch of the sound should be related to the spinner value -- higher pitch for larger value. Complicating things a bit is the fact that spinners have a dynamic range. I discussed with @jbphet, and we see 4 options for this sim:

(1) Disable UI sound for the 1.3 release (#77). Add it in some future release. Hope that NumberSpinner has proper sound by then.

(2) Use NumberSpinner "as is", with the default "button click" sound. While we can technically say the sim "supports sound", this option doesn't have a lot of value to the user, because doesn't tell them whether the value increased or decreased.

(3) Do a proper sound design/implementation for NumberSpinner. This will involve more people, take longer (probably weeks or months), but will benefit other sims that use NumberSpinner.

(4) Disable the default sounds for NumberSpinner and do something custom for RPAL. This is something that I could do myself, probably in an afternoon.

@kathy-phet @arouinfar How do you want to proceed? My recommendation is option (1), if you have the resources and it fits the timeline for publishing this sim. My second choice is option (4).

@arouinfar
Copy link

NumberSpinner is so integral to this sim. I do not think the default button sound provides particularly useful feedback for the NumberSpinner interaction. I would rather disable sound (1) than leave it as-is (2).

I strongly vote for option (3). I don't know the timeline for this particular sim, but If our goal as a project is to publish sims with a basic inclusive feature set, we need to invest time into common code as these sorts of issues arise. Otherwise, we will just end up kicking the can down the road (to probably just kick it again in the future).

I don't know if alt input is also on the table for this release, but supportsInteractiveDescription=true shows good coverage, though we would need to adjust the focus order.

@arouinfar arouinfar removed their assignment Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants