Removed un-used visible property and added jsdocs.

Author: photonstorm

src/animations/Animation.js

@@ -462,8 +462,6 @@ var Animation = new Class({
             });
         }
 
-        // console.table(frames);
-
         if (!Array.isArray(frames) || frames.length === 0)
         {
             return out;
@@ -480,23 +478,17 @@ var Animation = new Class({
                 continue;
             }
 
+            //  Could be an integer or a string
             var frame = GetValue(item, 'frame', 0);
 
+            //  The actual texture frame
             var textureFrame = textureManager.getFrame(key, frame);
 
             animationFrame = new Frame(key, frame, index, textureFrame);
 
             animationFrame.duration = GetValue(item, 'duration', 0);
             animationFrame.onUpdate = GetValue(item, 'onUpdate', null);
 
-            var visible = GetValue(item, 'visible', null);
-
-            if (visible !== null)
-            {
-                animationFrame.setVisible = true;
-                animationFrame.visible = visible;
-            }
-
             animationFrame.isFirst = (!prev);
 
             //  The previously created animationFrame

src/animations/AnimationFrame.js

@@ -1,156 +1,156 @@
 var Class = require('../utils/Class');
 
-//  Phaser.Animations.AnimationFrame
-
 var AnimationFrame = new Class({
 
     initialize:
 
     /**
-     * [description]
+     * A single frame in an Animation sequence.
+     *
+     * An AnimationFrame consists of a reference to the Texture it uses for rendering, references to other
+     * frames in the animation, and index data. It also has the ability to fire its own `onUpdate` callback
+     * and modify the animation timing.
+     *
+     * AnimationFrames are generated automatically by the Animation class.
      *
      * @class AnimationFrame
-     * @memberOf Phaser.Animations.AnimationFrame
+     * @memberOf Phaser.Animations
      * @constructor
      * @since 3.0.0
      *
-     * @param {undefined} textureKey - [description]
-     * @param {undefined} textureFrame - [description]
-     * @param {undefined} index - [description]
-     * @param {undefined} frame - [description]
+     * @param {string} textureKey - The key of the Texture this AnimationFrame uses.
+     * @param {string|integer} textureFrame - The key of the Frame within the Texture that this AnimationFrame uses.
+     * @param {integer} index - The index of this AnimationFrame within the Animation sequence.
+     * @param {Phaser.Textures.Frame} frame - A reference to the Texture Frame this AnimationFrame uses for rendering.
      */
     function AnimationFrame (textureKey, textureFrame, index, frame)
     {
-        //  The keys into the Texture Manager of the texture + frame this uses
-
         /**
-         * [description]
+         * The key of the Texture this AnimationFrame uses.
          *
          * @property {string} textureKey
+         * @since 3.0.0
          */
         this.textureKey = textureKey;
 
         /**
-         * [description]
+         * The key of the Frame within the Texture that this AnimationFrame uses.
          *
-         * @property {Phaser.Textures.Frame} textureFrame
+         * @property {string|integer} textureFrame
+         * @since 3.0.0
          */
         this.textureFrame = textureFrame;
 
-        //  The index of this frame within the Animation.frames array
-
         /**
-         * [description]
+         * The index of this AnimationFrame within the Animation sequence.
          *
          * @property {integer} index
+         * @since 3.0.0
          */
         this.index = index;
 
-        //  Texture Frame
-
         /**
-         * [description]
+         * A reference to the Texture Frame this AnimationFrame uses for rendering.
          *
          * @property {Phaser.Textures.Frame} frame
+         * @since 3.0.0
          */
         this.frame = frame;
 
-        //  Read-only
-
         /**
-         * [description]
+         * Is this the first frame in an animation sequence?
          *
          * @property {boolean} isFirst
          * @default false
+         * @readOnly
+         * @since 3.0.0
          */
         this.isFirst = false;
 
-        //  Read-only
-
         /**
-         * [description]
+         * Is this the last frame in an animation sequence?
          *
          * @property {boolean} isLast
          * @default false
+         * @readOnly
+         * @since 3.0.0
          */
         this.isLast = false;
 
-        //  The frame that comes before this one in the animation (if any)
-        //  Read-only
-
         /**
-         * [description]
+         * A reference to the AnimationFrame that comes before this one in the animation, if any.
          *
-         * @property {?[type]} prevFrame
+         * @property {?Phaser.Animations.AnimationFrame} prevFrame
          * @default null
+         * @readOnly
+         * @since 3.0.0
          */
         this.prevFrame = null;
 
-        //  The frame that comes after this one in the animation (if any)
-        //  Read-only
-
         /**
-         * [description]
+         * A reference to the AnimationFrame that comes after this one in the animation, if any.
          *
-         * @property {?[type]} nextFrame
+         * @property {?Phaser.Animations.AnimationFrame} nextFrame
          * @default null
+         * @readOnly
+         * @since 3.0.0
          */
         this.nextFrame = null;
 
-        //   Additional time (in ms) this frame should appear for - added onto the msPerFrame
-
         /**
-         * [description]
+         * Additional time (in ms) that this frame should appear for during playback.
+         * The value is added onto the msPerFrame set by the animation.
          *
          * @property {number} duration
          * @default 0
+         * @since 3.0.0
          */
         this.duration = 0;
 
-        //   What % through the animation progress is this frame?
-        //  Read-only
-
         /**
-         * [description]
+         * What % through the animation does this frame come?
+         * This value is generated when the animation is created and cached here.
          *
          * @property {number} progress
          * @default 0
+         * @readOnly
+         * @since 3.0.0
          */
         this.progress = 0;
 
-        //  Callback if this frame gets displayed
-
         /**
-         * [description]
+         * A frame specific callback, invoked if this frame gets displayed and the callback is set.
          *
-         * @property {?[type]} onUpdate
+         * @property {?function} onUpdate
          * @default null
+         * @since 3.0.0
          */
         this.onUpdate = null;
-
-        //  When this frame hits, set sprite.visible to this
-
-        /**
-         * [description]
-         *
-         * @property {boolean} setVisible
-         * @default false
-         */
-        this.setVisible = false;
-
-        this.visible = false;
     },
 
+    /**
+     * Generates a JavaScript object suitable for converting to JSON.
+     *
+     * @method Phaser.Animations.AnimationFrame#toJSON
+     * @since 3.0.0
+     * 
+     * @return {object} The AnimationFrame data.
+     */
     toJSON: function ()
     {
         return {
             key: this.textureKey,
             frame: this.textureFrame,
-            duration: this.duration,
-            visible: this.visible
+            duration: this.duration
         };
     },
 
+    /**
+     * Destroys this object by removing references to external resources and callbacks.
+     *
+     * @method Phaser.Animations.AnimationFrame#destroy
+     * @since 3.0.0
+     */
     destroy: function ()
     {
         this.frame = undefined;