Skip to content

Commit

Permalink
add "Voice:" to name response, #813
Browse files Browse the repository at this point in the history
  • Loading branch information
zepumph committed Jun 8, 2022
1 parent e445764 commit 728b223
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
3 changes: 3 additions & 0 deletions joist-strings_en.json
Original file line number Diff line number Diff line change
Expand Up @@ -363,6 +363,9 @@
"title": {
"value": "Voice"
},
"titlePattern": {
"value": "Voice: {{value}}"
},
"noVoicesAvailable": {
"value": "No voices available"
}
Expand Down
1 change: 1 addition & 0 deletions js/joistStrings.ts
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,7 @@ type StringsType = {
'writtenVariablesPattern': string;
'voice': {
'title': string;
'titlePattern': string;
'noVoicesAvailable': string;
};
'rate': {
Expand Down
3 changes: 3 additions & 0 deletions js/preferences/VoicingPanelSection.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ const voicingHintsString = joistStrings.a11y.preferences.tabs.audio.voicing.simV
const hintsMutedString = joistStrings.a11y.preferences.tabs.audio.voicing.simVoicingOptions.helpfulHints.disabledAlert;

const voiceLabelString = joistStrings.a11y.preferences.tabs.audio.voicing.customizeVoice.voice.title;
const voiceTitlePatternLabelString = joistStrings.a11y.preferences.tabs.audio.voicing.customizeVoice.voice.titlePattern;
const noVoicesAvailableString = joistStrings.a11y.preferences.tabs.audio.voicing.customizeVoice.voice.noVoicesAvailable;

const customizeVoiceExpandedString = joistStrings.a11y.preferences.tabs.audio.voicing.customizeVoice.expandedAlert;
Expand Down Expand Up @@ -431,6 +432,8 @@ class VoiceComboBox extends ComboBox {
listPosition: 'above',
accessibleName: voiceLabelString,

comboBoxVoicingNameResponsePattern: voiceTitlePatternLabelString,

// phet-io, opt out because we would need to instrument voices, but those could change between runtimes.
tandem: Tandem.OPT_OUT
}, options );
Expand Down

0 comments on commit 728b223

Please sign in to comment.