Timeline: Code cleanup. Fix bug with unfound property in serialization.

4 files changed, 6 insertions, 286 deletions

diff --git a/js/panels/Timeline/Collapser.js b/js/panels/Timeline/Collapser.js
index 32733b0a..88314c8b 100644
--- a/js/panels/Timeline/Collapser.js
+++ b/js/panels/Timeline/Collapser.js
@@ -179,24 +179,14 @@ var Montage = require("montage/core/core").Montage,
         
         prepareForDraw: {
                 value: function() {
-                        // Add a click listener to the label for expand/collapse
-                        /*
-                        if (this.isLabelClickable) {
-                                this.clicker.identifier = "collapserLabel";
-                                this.clicker.addEventListener("click", this, false);
-                        }
-                        */
+
                         // Get the original value of the overflow property:
                         this._origOverflowValue = window.getComputedStyle(this.myContent, null).getPropertyValue("overflow");
-                        console.log("collapser: this.isCollapsed " + this.isCollapsed)
                         if (this.isCollapsed === false) {
                                 this.myContent.style.height = "auto";
                         }
                         
-                        /*
-                         * Removed because of expense. This disables the feature of having the
-                         * component dynamically expand/collapse the content on init based on properties;
-                         * Now default state of component must be set in CSS.
+
                         // If the content area is supposed to start out collapsed:
                         if (this.isCollapsed) {
                                 this.myContent.style.height = "0px";
@@ -211,7 +201,6 @@ var Montage = require("montage/core/core").Montage,
                                 this.myContent.classList.remove(this.collapsedClass);
                                 this.clicker.classList.remove(this.collapsedClass);
                         }
-                        */
                 }
         },
         draw: {
diff --git a/js/panels/Timeline/Layer.reel/Layer.js b/js/panels/Timeline/Layer.reel/Layer.js
index 158eac00..81bd1867 100644
--- a/js/panels/Timeline/Layer.reel/Layer.js
+++ b/js/panels/Timeline/Layer.reel/Layer.js
@@ -398,11 +398,7 @@ var Layer = exports.Layer = Montage.create(Component, {
                         this.init();
                         
                 var that = this;
-                
-                        //this.positionCollapser = Collapser.create();
-                //this.transformCollapser = Collapser.create();
-                //this.styleCollapser = Collapser.create();
-                
+
                 // Make it editable!
                 this._layerEditable = Hintable.create();
                 this._layerEditable.element = this.titleSelector;
@@ -418,133 +414,12 @@ var Layer = exports.Layer = Montage.create(Component, {
                 this._layerEditable.editingClass = "editable2";
                 this._layerEditable.value = this.layerName;
                 this._layerEditable.needsDraw = true;
-                
-                // Change the markup into collapsible sections using the nifty Collapser component!
-                /*
-                this.mainCollapser = Collapser.create();
-                this.mainCollapser.clicker = this.clicker;
-                this.mainCollapser.myContent = this.myContent;
-                this.mainCollapser.contentHeight = 60;
-                this.myContent.style.height = "0px";
-            this.mainCollapser.element = this.myContent;
-            this.mainCollapser.isCollapsed = this.isMainCollapsed;
-            this.mainCollapser.isAnimated = true;
-            this.element.setAttribute("data-layerid", this.layerID);
-            // Bind the collapser's isToggling property to the isMainCollapsed property,
-            // so a change in one will affect the other.
-            Object.defineBinding(this.mainCollapser, "isToggling", {
-                                boundObject: this,
-                    boundObjectPropertyPath: "isMainCollapsed",
-                    oneway: false
-                        });
-                        */
+
             this.mainCollapser.clicker.addEventListener("click", this.handleMainCollapserClick.bind(this), false);
-            
-            /*
-            this.mainCollapser.labelClickEvent = function(boolBypass) {
-                                var newEvent = document.createEvent("CustomEvent");
-                                newEvent.initCustomEvent("layerEvent", false, true);
-                                newEvent.layerEventLocale = "content-main";
-                                newEvent.layerEventType = "labelClick";
-                                newEvent.layerID = that.layerID;
-                                newEvent.bypassAnimation = boolBypass;
-                                defaultEventManager.dispatchEvent(newEvent);
-                                that.isMainCollapsed = that.mainCollapser.isCollapsed;
-            }
-            */
-                //this.mainCollapser.needsDraw = true;
-/*
-            this.positionCollapser.clicker = this.clickerPosition;
-            this.positionCollapser.myContent = this.contentPosition;
-            this.positionCollapser.element = this.contentPosition;
-            this.positionCollapser.contentHeight = 40;
-            this.positionCollapser.isCollapsed = this.isPositionCollapsed;
-            this.positionCollapser.isAnimated = false;
-            // Bind the collapser's isToggling property to the isPositionCollapsed property,
-            // so a change in one will affect the other.
-            Object.defineBinding(this.positionCollapser, "isToggling", {
-                                boundObject: this,
-                    boundObjectPropertyPath: "isPositionCollapsed",
-                    oneway: false
-                        });
-                        */
             this.positionCollapser.clicker.addEventListener("click", this.handlePositionCollapserClick.bind(this), false);
-            
-            /*
-            this.positionCollapser.labelClickEvent = function(boolBypass) {
-                                var newEvent = document.createEvent("CustomEvent");
-                                newEvent.initCustomEvent("layerEvent", false, true);
-                                newEvent.layerEventLocale = "content-position";
-                                newEvent.layerEventType = "labelClick";
-                                newEvent.layerID = that.layerID;
-                                newEvent.bypassAnimation = boolBypass;
-                                defaultEventManager.dispatchEvent(newEvent);
-                                that.isPositionCollapsed = that.positionCollapser.isCollapsed;
-            }
-            */
-            //this.positionCollapser.needsDraw = true;
-    /*        
-            this.transformCollapser.clicker = this.clickerTransform;
-            this.transformCollapser.myContent = this.contentTransform;
-            this.transformCollapser.element = this.contentTransform;
-            this.transformCollapser.contentHeight = 100;
-            this.transformCollapser.isCollapsed = this.isTransformCollapsed;
-            this.transformCollapser.isAnimated = false;
-            // Bind the collapser's isToggling property to the isTransformCollapsed property,
-            // so a change in one will affect the other.
-            Object.defineBinding(this.transformCollapser, "isToggling", {
-                                boundObject: this,
-                    boundObjectPropertyPath: "isTransformCollapsed",
-                    oneway: false
-                        });
-                        */
             this.transformCollapser.clicker.addEventListener("click", this.handleTransformCollapserClick.bind(this), false);
-            
-            
-            /*
-            this.transformCollapser.labelClickEvent = function(boolBypass) {
-                                var newEvent = document.createEvent("CustomEvent");
-                                newEvent.initCustomEvent("layerEvent", false, true);
-                                newEvent.layerEventLocale = "content-transform";
-                                newEvent.layerEventType = "labelClick";
-                                newEvent.layerID = that.layerID;
-                                newEvent.bypassAnimation = boolBypass;
-                                defaultEventManager.dispatchEvent(newEvent);
-                                that.isTransformCollapsed = that.transformCollapser.isCollapsed;
-            }
-            */
-            //this.transformCollapser.needsDraw = true;
-            /*
-            this.styleCollapser.clicker = this.clickerStyle;
-            this.styleCollapser.myContent = this.contentStyle;
-            this.styleCollapser.element = this.contentStyle;
-            this.styleCollapser.isCollapsed = this.isStyleCollapsed;
-            this.styleCollapser.contentHeight = 0;
-            this.styleCollapser.isAnimated = false;
-            // Bind the collapser's isToggling property to the isStyleCollapsed property,
-            // so a change in one will affect the other.
-            Object.defineBinding(this.styleCollapser, "isToggling", {
-                                boundObject: this,
-                    boundObjectPropertyPath: "isStyleCollapsed",
-                    oneway: false
-                        });
-                        */
             this.styleCollapser.clicker.addEventListener("click", this.handleStyleCollapserClick.bind(this), false);
-            
-            
-            /*
-            this.styleCollapser.labelClickEvent = function(boolBypass) {
-                                var newEvent = document.createEvent("CustomEvent");
-                                newEvent.initCustomEvent("layerEvent", false, true);
-                                newEvent.layerEventLocale = "content-style";
-                                newEvent.layerEventType = "labelClick";
-                                newEvent.layerID = that.layerID;
-                                newEvent.bypassAnimation = boolBypass;
-                                defaultEventManager.dispatchEvent(newEvent);
-                                that.isStyleCollapsed = that.styleCollapser.isCollapsed;
-            }
-            */
-            //this.styleCollapser.needsDraw = true;
+
 
             // Add event listeners to add and delete style buttons
             this.buttonAddStyle.identifier = "addStyle";
@@ -561,27 +436,6 @@ var Layer = exports.Layer = Montage.create(Component, {
     },
     draw: {
         value: function() {