Skip to content

Commit

Permalink
TODOs for #140
Browse files Browse the repository at this point in the history
  • Loading branch information
pixelzoom committed Aug 31, 2022
1 parent 70adb47 commit ec4dcd1
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 0 deletions.
2 changes: 2 additions & 0 deletions js/common/view/DipoleDirectionRadioButtonGroup.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,14 @@ class DipoleDirectionRadioButtonGroup extends AquaRadioButtonGroup {
}, MPConstants.AQUA_RADIO_BUTTON_OPTIONS, options );

// d+ -> d-
//TODO https://github.com/phetsims/molecule-polarity/issues/140 support for dynamic locale
const positiveToNegativeString = StringUtils.fillIn( moleculePolarityStrings.pattern.dipoleDirection, {
from: moleculePolarityStrings.deltaPlus,
to: moleculePolarityStrings.deltaMinus
} );

// d- -> d+
//TODO https://github.com/phetsims/molecule-polarity/issues/140 support for dynamic locale
const negativeToPositiveString = StringUtils.fillIn( moleculePolarityStrings.pattern.dipoleDirection, {
from: moleculePolarityStrings.deltaMinus,
to: moleculePolarityStrings.deltaPlus
Expand Down
1 change: 1 addition & 0 deletions js/common/view/PartialChargeNode.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ class PartialChargeNode extends Node {
if ( partialChargeVisible ) {

// d+ or d-
//TODO https://github.com/phetsims/molecule-polarity/issues/140 support for dynamic locale
textNode.text = ( partialCharge > 0 ) ? moleculePolarityStrings.deltaPlus : moleculePolarityStrings.deltaMinus;

// size proportional to bond dipole magnitude
Expand Down
1 change: 1 addition & 0 deletions js/realmolecules/model/RealMolecule.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ import moleculePolarity from '../../moleculePolarity.js';

class RealMolecule extends PhetioObject {

//TODO https://github.com/phetsims/molecule-polarity/issues/140 support for dynamic locale
/**
* @param {string} symbol - chemical symbol of the molecule
* @param {string} fullName - full name of the molecule
Expand Down
2 changes: 2 additions & 0 deletions js/realmolecules/view/RealMoleculeViewer.js
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,8 @@ class RealMoleculeViewer extends Node {

// unlink not needed
moleculeProperty.link( molecule => {

//TODO https://github.com/phetsims/molecule-polarity/issues/140 support for dynamic locale
moleculeText.text = StringUtils.fillIn( moleculePolarityStrings.pattern.symbolName, {
symbol: molecule.symbol,
name: molecule.fullName
Expand Down
1 change: 1 addition & 0 deletions js/realmolecules/view/RealMoleculesComboBox.js
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ class RealMoleculesComboBox extends ComboBox {
function createItem( molecule ) {
assert && assert( molecule instanceof RealMolecule, 'invalid molecule' );

//TODO https://github.com/phetsims/molecule-polarity/issues/140 support for dynamic locale
const text = StringUtils.fillIn( moleculePolarityStrings.pattern.symbolName, {
symbol: molecule.symbol,
name: molecule.fullName
Expand Down

0 comments on commit ec4dcd1

Please sign in to comment.