Skip to content

Commit

Permalink
Rename allUrl to allURL, phetsims/number-pairs#22
Browse files Browse the repository at this point in the history
  • Loading branch information
pixelzoom committed Dec 3, 2024
1 parent 54ff87d commit e9d5849
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 11 deletions.
6 changes: 3 additions & 3 deletions js/common/model/NumberSuiteCommonPreferences.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@ class NumberSuiteCommonPreferences {
public readonly secondLocaleStringsProperty: TReadOnlyProperty<SecondLocaleStrings>;

// URL to the {REPO}_all.html file for this simulation.
public readonly allUrl: string;
public readonly allURL: string;

public constructor( allUrl: string ) {
public constructor( allURL: string ) {

// if a valid second locale was provided via a query parameter, display the second locale on sim startup
this.showSecondLocaleProperty = new BooleanProperty( !!NumberSuiteCommonQueryParameters.secondLocale );
Expand All @@ -76,7 +76,7 @@ class NumberSuiteCommonPreferences {
return phet.chipper.strings[ secondLocale ];
} );

this.allUrl = allUrl;
this.allURL = allURL;
}

public dispose(): void {
Expand Down
2 changes: 1 addition & 1 deletion js/common/view/NumberSuiteCommonPreferencesNode.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ export default class NumberSuiteCommonPreferencesNode extends HBox {
align: 'top'
}, providedOptions );

const secondLanguageControl = new SecondLanguageControl( preferences, preferences.allUrl, utteranceQueue, {
const secondLanguageControl = new SecondLanguageControl( preferences, preferences.allURL, utteranceQueue, {
visible: options.secondLanguageControlEnabled
} );

Expand Down
14 changes: 7 additions & 7 deletions js/common/view/SecondLanguageControl.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,12 @@ export default class SecondLanguageControl extends VBox {

/**
* @param secondLanguageProperties
* @param allUrl - URL to the {REPO}_all.html file for this simulation.
* @param allURL - URL to the {REPO}_all.html file for this simulation.
* @param utteranceQueue
* @param [providedOptions]
*/
public constructor( secondLanguageProperties: SecondLanguageProperties,
allUrl: string,
allURL: string,
utteranceQueue: NumberSuiteCommonUtteranceQueue,
providedOptions?: SecondLanguageControlOptions ) {

Expand Down Expand Up @@ -75,7 +75,7 @@ export default class SecondLanguageControl extends VBox {
} );

// Additional description that is visible when the Second Language control is disabled.
const additionalDescriptionNode = new AdditionalDescriptionNode( !preferencesControl.enabled, allUrl );
const additionalDescriptionNode = new AdditionalDescriptionNode( !preferencesControl.enabled, allURL );

// Control for choosing a second language and associated voice
const languageAndVoiceControl = new LanguageAndVoiceControl(
Expand Down Expand Up @@ -125,9 +125,9 @@ class AdditionalDescriptionNode extends VBox {

/**
* @param visible
* @param allUrl - URL to the {REPO}_all.html file for this simulation.
* @param allURL - URL to the {REPO}_all.html file for this simulation.
*/
public constructor( visible: boolean, allUrl: string ) {
public constructor( visible: boolean, allURL: string ) {

const toDisplayASecondLanguageText = new RichText( NumberSuiteCommonStrings.toDisplayASecondLanguageDescriptionStringProperty, {
font: new PhetFont( 12 ),
Expand All @@ -136,10 +136,10 @@ class AdditionalDescriptionNode extends VBox {

// If links are not allowed, show the URL as plain text.
const urlStringProperty = new DerivedProperty( [ allowLinksProperty ],
allowLinks => allowLinks ? `<a href="{{url}}">${allUrl}</a>` : allUrl
allowLinks => allowLinks ? `<a href="{{url}}">${allURL}</a>` : allURL
);
const urlText = new RichText( urlStringProperty, {
links: { url: allUrl },
links: { url: allURL },
font: new PhetFont( 12 ),
maxWidth: PreferencesDialog.CONTENT_MAX_WIDTH
} );
Expand Down

0 comments on commit e9d5849

Please sign in to comment.