Skip to content

Commit

Permalink
Removed extraneous NumberLine option and confusing doc language
Browse files Browse the repository at this point in the history
  • Loading branch information
arnabp authored and jbphet committed Apr 1, 2020
1 parent c9b93e9 commit 392f5d6
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions js/common/model/NumberLine.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ define( require => {
// {Range} - range of values to be displayed
initialDisplayedRange: new Range( -10, 10 ),

// {boolean} - whether tick marks should be initially displayed
// {boolean} - whether point labels should initially be shown
labelsInitiallyVisible: false,

// {boolean} - whether tick marks should be initially displayed
Expand All @@ -60,9 +60,6 @@ define( require => {
// {boolean} - whether absolute value indicators should be initially displayed
absoluteValuesInitiallyVisible: false,

// {boolean} - whether point labels should initially be shown
pointLabelsInitiallyShown: false,

// {number} - The width and height values used when projecting the number line into model space. The default
// values are pretty arbitrary and at least one of these will generally need to be set. However, if the number
// line is only ever shown in one orientation, the value corresponding to the other orientation can be left at
Expand All @@ -84,7 +81,7 @@ define( require => {
// @public {Property<Range>} - the range of values that should be displayed to the user
this.displayedRangeProperty = new Property( options.initialDisplayedRange );

// @public {BooleanProperty} - controls whether tick marks should be displayed to the user
// @public {BooleanProperty} - controls whether point labels are displayed to the user
this.labelsVisibleProperty = new BooleanProperty( options.labelsInitiallyVisible );

// @public {BooleanProperty} - controls whether tick marks should be displayed to the user
Expand All @@ -96,9 +93,6 @@ define( require => {
// @public {BooleanProperty} - controls whether tick marks should be displayed to the user
this.oppositesVisibleProperty = new BooleanProperty( false );

// @public {BooleanProperty} - controls whether point labels are displayed to the user
this.showPointLabels = new BooleanProperty( options.pointLabelsInitiallyShown );

// @public (read-only) {ObservableArray<NumberLinePoint>} - array of points on this number line
this.residentPoints = new ObservableArray();

Expand Down Expand Up @@ -359,7 +353,6 @@ define( require => {
this.labelsVisibleProperty.reset();
this.tickMarksVisibleProperty.reset();
this.oppositesVisibleProperty.reset();
this.showPointLabels.reset();
this.addInitialPoints();
}
}
Expand Down

0 comments on commit 392f5d6

Please sign in to comment.