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

1 files changed, 4 insertions, 160 deletions

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() {
-/*
-                // Coordinate the collapsers
-                this.log('layer.js draw')
-            if (this.mainCollapser.isCollapsed !== this.isMainCollapsed) {
-                this.log('layer.js draw: this.animateCollapser ' + this.animateCollapser)
-                this.mainCollapser.bypassAnimation = this.animateCollapser;
-                this.mainCollapser.toggle();
-            }
-            if (this.positionCollapser.isCollapsed !== this.isPositionCollapsed) {
-                this.positionCollapser.bypassAnimation = this.animateCollapser;
-                this.positionCollapser.toggle();
-            }
-            if (this.transformCollapser.isCollapsed !== this.isTransformCollapsed) {
-                this.transformCollapser.bypassAnimation = this.animateCollapser;
-                this.transformCollapser.toggle();
-            }
-            if (this.styleCollapser.isCollapsed !== this.isStyleCollapsed) {
-                this.styleCollapser.bypassAnimation = this.animateCollapser;
-                this.styleCollapser.toggle();
-            }
-            */
             if (this.isSelected) {
                 this.element.classList.add("selected");
             } else {
@@ -604,17 +458,7 @@ var Layer = exports.Layer = Montage.create(Component, {
                 this.titleSelector = this.label.querySelector(".collapsible-label");
                 this.buttonAddStyle = this.element.querySelector(".button-add");
                 this.buttonDeleteStyle = this.element.querySelector(".button-delete");
-                
-                /*
-                this.clicker = this.element.querySelector(".collapsible-clicker");
-                this.myContent = this.element.querySelector(".content-layer");
-                this.clickerPosition = this.element.querySelector(".clicker-position");
-                this.contentPosition = this.element.querySelector(".content-position");
-                this.clickerTransform = this.element.querySelector(".clicker-transform");
-                this.contentTransform = this.element.querySelector(".content-transform");
-                this.clickerStyle = this.element.querySelector(".clicker-style");
-                this.contentStyle = this.element.querySelector(".content-style");
-                */
+
                 }
         },
     selectLayer:{