-
Notifications
You must be signed in to change notification settings - Fork 6
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
Delete joistVoicingUtteranceQueue and replace with voicingVisibleProperty #800
Comments
At this time the only usage of |
It isn't working quite yet but I expect that it will. We need to do phetsims/scenery#1403 to verify that |
|
This should be reviewed as part of phetsims/scenery#1300, assigning to @zepumph as part of that. And thanks! |
One improvement is to rename We need to update voicingVisible somehow in setAccessibleViewsVisible. We also need explicit control over how voicingVisible is enabled for the Toolbar and topLayer (dialogs) we thought it would be good to have two functions for each of those so that setting is clear. Also, the call to Back to me for these things. |
Ok sounds good. Let me know if I can assist at all. |
Now that
voicingVisibleProperty
is ready for use I think it can be used instead of thejoistVoicingUtteranceQueue
. ThejoistVoicingUtteranceQueue
is problematic becausepriorityProperty
doesn't work well with multiple UtteranceQueues (see #752 (comment)).The text was updated successfully, but these errors were encountered: