Skip to content

Commit

Permalink
Renamed "check box" to "checkbox" in comments and documentation, see p…
Browse files Browse the repository at this point in the history
  • Loading branch information
samreid committed Jan 12, 2018
1 parent 0de281b commit 85bb62b
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 13 deletions.
6 changes: 3 additions & 3 deletions js/common/view/HookesLawIconFactory.js
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ define( function( require ) {
},

/**
* Creates the content for a vector check box, consisting of text and an arrow.
* Creates the content for a vector checkbox, consisting of text and an arrow.
*
* @param {Node} textNode - text, positioned to the left of the vector
* @param {Object} [options]
Expand All @@ -190,7 +190,7 @@ define( function( require ) {
createVectorCheckboxContent: function( textNode, options ) {

options = _.extend( {
maxTextWidth: textNode.width, // width of the max text used to label a vector check box
maxTextWidth: textNode.width, // width of the max text used to label a vector checkbox
spacing: 10, // {number} space between text and vector
arrowLength: 30, // {number}
arrowDirection: 'right', // {string} direction that the vector points, 'left' or 'right',
Expand Down Expand Up @@ -223,7 +223,7 @@ define( function( require ) {
},

/**
* Creates the icon for the equilibrium position check box, consisting of text and a vertical dashed line.
* Creates the icon for the equilibrium position checkbox, consisting of text and a vertical dashed line.
* @returns {Node}
* @public
*/
Expand Down
6 changes: 3 additions & 3 deletions js/energy/view/EnergyVisibilityControls.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ define( function( require ) {
new Text( forcePlotString, HookesLawConstants.CONTROL_TEXT_OPTIONS ),
HookesLawConstants.RADIO_BUTTON_OPTIONS );

// energy check box, enabled when "Force Plot" radio button is selected
// energy checkbox, enabled when "Force Plot" radio button is selected
var energyIcon = new HBox( {
children: [
new Text( energyString, HookesLawConstants.CONTROL_TEXT_OPTIONS ),
Expand All @@ -70,7 +70,7 @@ define( function( require ) {
energyCheckbox.enabled = ( graph === 'forceXY' );
} );

// other check boxes
// other checkboxes
var appliedForceCheckbox = new Checkbox(
HookesLawIconFactory.createVectorCheckboxContent( new Text( appliedForceString, HookesLawConstants.CONTROL_TEXT_OPTIONS ), {
arrowFill: HookesLawColors.APPLIED_FORCE
Expand Down Expand Up @@ -116,7 +116,7 @@ define( function( require ) {
energyBarRadioButton,
energyXYRadioButton,
forceXYRadioButton,
// "Energy" check box indented below "Force Plot" radio button
// "Energy" checkbox indented below "Force Plot" radio button
new HBox( { children: [ new HStrut( 25 ), energyCheckbox ] } ),
new HSeparator( maxControlWidth, HookesLawConstants.SEPARATOR_OPTIONS ),
appliedForceCheckbox,
Expand Down
8 changes: 4 additions & 4 deletions js/intro/view/IntroVisibilityControls.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,15 @@ define( function( require ) {

options = _.extend( _.clone( HookesLawConstants.VISIBILITY_PANEL_OPTIONS ), options );

// text labels on the vector check boxes
// text labels on the vector checkboxes
var appliedForceTextNode = new Text( appliedForceString, HookesLawConstants.CONTROL_TEXT_OPTIONS );
var springForceTextNode = new Text( springForceString, HookesLawConstants.CONTROL_TEXT_OPTIONS );
var displacementTextNode = new Text( displacementString, HookesLawConstants.CONTROL_TEXT_OPTIONS );
var maxTextWidth = _.maxBy( [ appliedForceTextNode, springForceTextNode, displacementTextNode ], function( node ) {
return node.width;
} ).width;

// vector check boxes, with left-aligned vector icons
// vector checkboxes, with left-aligned vector icons
var minSpacing = 10;
var appliedForceCheckbox = new Checkbox(
HookesLawIconFactory.createVectorCheckboxContent( appliedForceTextNode, {
Expand All @@ -68,7 +68,7 @@ define( function( require ) {
properties.displacementVectorVisibleProperty,
HookesLawConstants.CHECK_BOX_OPTIONS );

// other check boxes
// other checkboxes
var equilibriumPositionCheckbox = new Checkbox(
HookesLawIconFactory.createEquilibriumPositionCheckboxContent(),
properties.equilibriumPositionVisibleProperty,
Expand All @@ -78,7 +78,7 @@ define( function( require ) {
properties.valuesVisibleProperty,
HookesLawConstants.CHECK_BOX_OPTIONS );

// 'Values' check box pertains to vectors, so enable that check box only if one or more of the vectors is selected.
// 'Values' checkbox pertains to vectors, so enable that checkbox only if one or more of the vectors is selected.
Property.multilink(
[ properties.appliedForceVectorVisibleProperty, properties.springForceVectorVisibleProperty, properties.displacementVectorVisibleProperty ],
function( appliedForceVectorVisible, springForceVectorVisible, displacementVectorVisible ) {
Expand Down
6 changes: 3 additions & 3 deletions js/systems/view/SystemsVisibilityControls.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ define( function( require ) {

options = _.extend( _.clone( HookesLawConstants.VISIBILITY_PANEL_OPTIONS ), options );

// vector check boxes
// vector checkboxes
var appliedForceCheckbox = new Checkbox(
HookesLawIconFactory.createVectorCheckboxContent( new Text( appliedForceString, HookesLawConstants.CONTROL_TEXT_OPTIONS ),
{ arrowFill: HookesLawColors.APPLIED_FORCE } ),
Expand All @@ -61,7 +61,7 @@ define( function( require ) {
properties.displacementVectorVisibleProperty,
HookesLawConstants.CHECK_BOX_OPTIONS );

// other check boxes
// other checkboxes
var equilibriumPositionCheckbox = new Checkbox(
HookesLawIconFactory.createEquilibriumPositionCheckboxContent(),
properties.equilibriumPositionVisibleProperty,
Expand Down Expand Up @@ -126,7 +126,7 @@ define( function( require ) {
spacing: 5
} );

// 'Values' check box pertains to vectors, so enable that check box only if one or more of the vectors is selected.
// 'Values' checkbox pertains to vectors, so enable that checkbox only if one or more of the vectors is selected.
Property.multilink(
[ properties.appliedForceVectorVisibleProperty, properties.springForceVectorVisibleProperty, properties.displacementVectorVisibleProperty ],
function( appliedForceVectorVisible, springForceVectorVisible, displacementVectorVisible ) {
Expand Down

0 comments on commit 85bb62b

Please sign in to comment.