From 52ca1c95d6f000503f44fa5f3139557e770a7773 Mon Sep 17 00:00:00 2001
From: Valerio Virgillito
Date: Tue, 12 Jun 2012 15:12:52 -0700
Subject: hack fix for the PI section throwing errors when dom is not ready to
 draw

Signed-off-by: Valerio Virgillito <valerio@motorola.com>
---
 js/controllers/selection-controller.js | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

(limited to 'js/controllers')

diff --git a/js/controllers/selection-controller.js b/js/controllers/selection-controller.js
index 9ae1cfc9..6d982e69 100755
--- a/js/controllers/selection-controller.js
+++ b/js/controllers/selection-controller.js
@@ -34,6 +34,7 @@ exports.SelectionController = Montage.create(Component, {
             }
 
             if(this._currentDocument && this._currentDocument.currentView === "design") {
+                this.ignoreFlag = false;
                 this._currentDocument.model._selection = this.application.ninja.selectedElements;
 //                this._currentDocument.model.selectionContainer = this.application.ninja._currentSelectedContainer;
             }
@@ -50,6 +51,10 @@ exports.SelectionController = Montage.create(Component, {
         }
     },
 
+    ignoreFlag: {
+        value: true
+    },
+
     _selectedElements: {
         value: null
     },
@@ -91,10 +96,14 @@ exports.SelectionController = Montage.create(Component, {
 
     handleChange: {
         value: function() {
-            if(this.currentDocument && this.currentDocument.model.getProperty("domContainer")) {
-                if(this.currentDocument.model.getProperty("domContainer") !== null) {
-                    this.executeSelectElement();
+            if(this.ignoreFlag) {
+                if(this.currentDocument && this.currentDocument.model.getProperty("domContainer")) {
+                    if(this.currentDocument.model.getProperty("domContainer") !== null) {
+                        this.executeSelectElement();
+                    }
                 }
+            } else {
+                this.ignoreFlag = true;
             }
         }
     },
-- 
cgit v1.2.3


From a75947d69f571d723552f926f94d195514a01cbd Mon Sep 17 00:00:00 2001
From: Valerio Virgillito
Date: Tue, 12 Jun 2012 23:04:07 -0700
Subject: fixed document switching issues

Signed-off-by: Valerio Virgillito <valerio@motorola.com>
---
 js/controllers/selection-controller.js | 31 ++++---------------------------
 1 file changed, 4 insertions(+), 27 deletions(-)

(limited to 'js/controllers')

diff --git a/js/controllers/selection-controller.js b/js/controllers/selection-controller.js
index 6d982e69..b327bea4 100755
--- a/js/controllers/selection-controller.js
+++ b/js/controllers/selection-controller.js
@@ -34,13 +34,14 @@ exports.SelectionController = Montage.create(Component, {
             }
 
             if(this._currentDocument && this._currentDocument.currentView === "design") {
-                this.ignoreFlag = false;
                 this._currentDocument.model._selection = this.application.ninja.selectedElements;
-//                this._currentDocument.model.selectionContainer = this.application.ninja._currentSelectedContainer;
             }
 
             this._currentDocument = value;
 
+            if(this._currentDocument && this._currentDocument.currentView === "design") {
+                this.selectedElements = this._currentDocument.model.selection;
+            }
             /*
             if(!value) {
             } else if(this._currentDocument.currentView === "design") {
@@ -51,10 +52,6 @@ exports.SelectionController = Montage.create(Component, {
         }
     },
 
-    ignoreFlag: {
-        value: true
-    },
-
     _selectedElements: {
         value: null
     },
@@ -64,21 +61,17 @@ exports.SelectionController = Montage.create(Component, {
             return this._selectedElements;
         },
         set: function(value) {
-            if(this.currentDocument && this.currentDocument.currentView === "code") return;
 
             if(value) {
                 this._selectedElements = value;
 
                 this.application.ninja.selectedElements = this._selectedElements;
-//                this.application.ninja._currentSelectedContainer = this._selectionContainer = this.application.ninja.currentDocument.model.documentRoot;
 
                 if(this._selectedElements.length === 0) {
                     this.executeSelectElement();
                 } else {
                     this.executeSelectElement(this._selectedElements);
                 }
-
-
             }
         }
     },
@@ -89,22 +82,6 @@ exports.SelectionController = Montage.create(Component, {
             this.eventManager.addEventListener("elementsRemoved", this, false);
             this.eventManager.addEventListener("elementReplaced", this, false);
             this.eventManager.addEventListener("selectAll", this, false);
-
-            this.addPropertyChangeListener("currentDocument.model.domContainer", this);
-        }
-    },
-
-    handleChange: {
-        value: function() {
-            if(this.ignoreFlag) {
-                if(this.currentDocument && this.currentDocument.model.getProperty("domContainer")) {
-                    if(this.currentDocument.model.getProperty("domContainer") !== null) {
-                        this.executeSelectElement();
-                    }
-                }
-            } else {
-                this.ignoreFlag = true;
-            }
         }
     },
 
@@ -134,7 +111,7 @@ exports.SelectionController = Montage.create(Component, {
         value: function(event) {
             var selected = [], childNodes = [], self = this;
 
-            childNodes = this.application.ninja.currentDocument.model.documentRoot.childNodes;
+            childNodes = this.currentDocument.model.domContainer.childNodes;
             childNodes = Array.prototype.slice.call(childNodes, 0);
             childNodes.forEach(function(item) {
                 if(self.isNodeTraversable(item)) {
-- 
cgit v1.2.3