Cleanup in Magnet, see #86

js/faradays-law/model/FaradaysLawModel.js

@@ -68,7 +68,7 @@ define( function( require ) {
     this.magnetMovingDirection = null; // TODO: should be enum
 
     // @public - the Voltmeter
-    this.voltmeter = new Voltmeter( this, tandem.createTandem( 'voltmeterModel' ) );
+    this.voltmeter = new Voltmeter( this, tandem.createTandem( 'voltmeter' ) );
 
     // If the magnet intersects the top coil area when the top coil is shown, then reset the magnet.
     this.showTopCoilProperty.link( function( showTopCoil ) {

js/faradays-law/model/Magnet.js

@@ -10,15 +10,13 @@ define( function( require ) {
   'use strict';
 
   // modules
+  var BooleanProperty = require( 'AXON/BooleanProperty' );
   var faradaysLaw = require( 'FARADAYS_LAW/faradaysLaw' );
   var inherit = require( 'PHET_CORE/inherit' );
   var Property = require( 'AXON/Property' );
   var TVector2 = require( 'DOT/TVector2' );
   var Vector2 = require( 'DOT/Vector2' );
 
-  // phet-io modules
-  var TBoolean = require( 'ifphetio!PHET_IO/types/TBoolean' );
-
   /**
    * @param {number} x - x position of magnet
    * @param {number} y - y position of magnet
@@ -29,31 +27,37 @@ define( function( require ) {
    */
   function Magnet( x, y, width, height, tandem ) {
 
+    // @public {number} - width of the magnet
     this.width = width;
+
+    // @public {number} - height of the magnet
     this.height = height;
 
+    // @public {Property.<Vector2>} - position of the magnet
     this.positionProperty = new Property( new Vector2( x, y ), {
       tandem: tandem.createTandem( 'positionProperty' ),
       phetioValueType: TVector2
     } );
 
-    // is the magnet flipped?
-    this.flippedProperty = new Property( false, {
-      tandem: tandem.createTandem( 'flippedProperty' ),
-      phetioValueType: TBoolean
+    // @public {BooleanProperty} - true if the magnet is flipped
+    this.flippedProperty = new BooleanProperty( false, {
+      tandem: tandem.createTandem( 'flippedProperty' )
     } );
 
-    // show field lines for magnet
-    this.showFieldLinesProperty = new Property( false, {
-      tandem: tandem.createTandem( 'showFieldLinesProperty' ),
-      phetioValueType: TBoolean
+    // @public {BooleanProperty} - show field lines for magnet
+    this.showFieldLinesProperty = new BooleanProperty( false, {
+      tandem: tandem.createTandem( 'showFieldLinesProperty' )
     } );
   }
 
   faradaysLaw.register( 'Magnet', Magnet );
 
   return inherit( Object, Magnet, {
 
+    /**
+     * Restore the initial conditions
+     * @public
+     */
     reset: function() {
       this.positionProperty.reset();
       this.flippedProperty.reset();

js/faradays-law/view/Aligner.js

@@ -19,7 +19,7 @@ define( function( require ) {
 
   /**
    *
-   * @param {FaradayslawModel} model
+   * @param {FaradaysLawModel} model
    * @param {Object} bottomCoilEndRelativePositions
    * @param {Object} topCoilEndRelativePositions
    * @constructor

js/faradays-law/view/FaradaysLawScreenView.js

@@ -25,7 +25,7 @@ define( function( require ) {
   var VoltmeterWiresNode = require( 'FARADAYS_LAW/faradays-law/view/VoltmeterWiresNode' );
 
   /**
-   * @param {gameModel} model - Faraday's Law simulation model object
+   * @param {FaradaysLawModel} model - Faraday's Law simulation model object
    * @param {Tandem} tandem
    * @constructor
    */

js/faradays-law/view/MagnetFieldLines.js

@@ -132,7 +132,6 @@ define( function( require ) {
     this.addChild( bottomLines );
 
     magnet.showFieldLinesProperty.linkAttribute( this, 'visible' );
-
   }
 
   faradaysLaw.register( 'MagnetFieldLines', MagnetFieldLines );

js/faradays-law/view/MagnetNodeWithField.js

@@ -30,7 +30,6 @@ define( function( require ) {
   };
 
   /**
-   *
    * @param {FaradaysLawModel} model - 'Faradays Law' simulation model
    * @param {Tandem} tandem - this node is not instrumented but the input listener is
    * @constructor