Skip to content

Commit

Permalink
visibility annotation, see #37
Browse files Browse the repository at this point in the history
  • Loading branch information
mbarlow12 committed Mar 2, 2018
1 parent ffcfaba commit c2b2a8b
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 0 deletions.
2 changes: 2 additions & 0 deletions js/model/ISLCObject.js
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,8 @@ define( function( require ) {

/**
* Calculate radius for the object - must be implemented in subtypes.
*
* @public
*/
calculateRadius: function() {
throw new Error( 'calculateRadius must be implemented in descendent types' );
Expand Down
2 changes: 2 additions & 0 deletions js/view/ISLCForceArrowNode.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,9 @@ define( function( require ) {
arrowOptions.fill = options.arrowFill;
arrowOptions.tandem = options.tandem.createTandem( 'arrow' );

// @private
this.arrow = new ArrowNode( 0, -options.forceArrowHeight, 200, -options.forceArrowHeight, arrowOptions );

Node.call( this, {} );
this.addChild( this.arrowText );
this.addChild( this.arrow );
Expand Down
1 change: 1 addition & 0 deletions js/view/ISLCObjectControlPanel.js
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,7 @@ define( function( require ) {

Panel.call( this, numberControl, panelOptions );

// @protected - used in Accessibility.js
this.groupFocusHighlight = new GroupFocusHighlightFromNode( this, {
useLocalBounds: true,
dilationCoefficient: 3.7,
Expand Down
5 changes: 5 additions & 0 deletions js/view/ISLCObjectNode.js
Original file line number Diff line number Diff line change
Expand Up @@ -321,6 +321,8 @@ define( function( require ) {

/**
* Redraws the white radial gradient for the object - must be implemented in subtypes.
*
* @public
*/
updateGradient: function() {
throw new Error( 'Update gradient must be implemented in subtypes.' );
Expand All @@ -329,6 +331,7 @@ define( function( require ) {
/**
* Sets whether or not the readouts above the arrows use scientific notation in their display of the value.
*
* @public
* @param {boolean} useScientificNotation
*/
setReadoutsInScientificNotation: function( useScientificNotation ) {
Expand All @@ -340,6 +343,8 @@ define( function( require ) {

/**
* Updates the radius, arrow length & direction, force readout, and the visible puller image.
*
* @public
*/
redrawForce: function() {
this.objectCircle.setRadius( this.modelViewTransform.modelToViewDeltaX( this.objectModel.radiusProperty.get() ) );
Expand Down

0 comments on commit c2b2a8b

Please sign in to comment.