Merge pull request #34 from ericguzman/TreeComponents

Tree components - updating the ninja branch/leaf
author: Valerio Virgillito 2012-02-13 15:08:39 -0800
committer: Valerio Virgillito 2012-02-13 15:08:39 -0800
commit: 24b721d49dd20a8ae24198a7dd61dce8838083d9 (patch)
tree: 427fc0008ced60bff67b41a165164da491777d97 /js
parent: 7b0910720a6979bf41c216c5072d5c7a91e6d0b7 (diff)
parent: 85afed8ed707ce20cd28c3139680dfef6fd4b23d (diff)
download: ninja-24b721d49dd20a8ae24198a7dd61dce8838083d9.tar.gz
4 files changed, 38 insertions, 10 deletions
diff --git a/js/components/treeview/ninja-branch.reel/ninja-branch.html b/js/components/treeview/ninja-branch.reel/ninja-branch.html
index 4d557a94..13f1c575 100644
--- a/js/components/treeview/ninja-branch.reel/ninja-branch.html
+++ b/js/components/treeview/ninja-branch.reel/ninja-branch.html
@@ -110,7 +110,8 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
            "name": "Repetition",
            "properties": {
                "element": { "#": "branchList" },
-                "contentController": {"@": "arrayController" }
+                "contentController": {"@": "arrayController" },
+                "selectionEnabled": true
            }
        },
diff --git a/js/components/treeview/ninja-branch.reel/ninja-branch.js b/js/components/treeview/ninja-branch.reel/ninja-branch.js
index 6b9ebb10..0d1a1d0e 100644
--- a/js/components/treeview/ninja-branch.reel/ninja-branch.js
+++ b/js/components/treeview/ninja-branch.reel/ninja-branch.js
@@ -5,7 +5,6 @@
 </copyright> */
 var Montage = require("montage").Montage,
-    Component = require("montage/ui/component").Component,
    TreeNode = require("js/components/treeview/tree-node").TreeNode;
 var Branch = exports.Branch = Montage.create(TreeNode, {
diff --git a/js/components/treeview/ninja-leaf.reel/ninja-leaf.html b/js/components/treeview/ninja-leaf.reel/ninja-leaf.html
index bd8e8acd..310274bc 100644
--- a/js/components/treeview/ninja-leaf.reel/ninja-leaf.html
+++ b/js/components/treeview/ninja-leaf.reel/ninja-leaf.html
@@ -37,7 +37,7 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
 </head>
 <body>
    <div id="leaf" class="leaf">
-        <div id="label" class="leaf-label"></div>
+        <div id="label" class="leaf-label" draggable="true"></div>
    </div>
 </body>
 </html>
 \ No newline at end of file
diff --git a/js/components/treeview/ninja-leaf.reel/ninja-leaf.js b/js/components/treeview/ninja-leaf.reel/ninja-leaf.js
index bd566b26..0b7a171e 100644
--- a/js/components/treeview/ninja-leaf.reel/ninja-leaf.js
+++ b/js/components/treeview/ninja-leaf.reel/ninja-leaf.js
@@ -5,7 +5,6 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
 </copyright> */
 var Montage   = require("montage/core/core").Montage,
-    Component = require("montage/ui/component").Component;
    TreeNode = require("js/components/treeview/tree-node").TreeNode;
 exports.Leaf = Montage.create(TreeNode, {
@@ -14,19 +13,38 @@ exports.Leaf = Montage.create(TreeNode, {
    },
    templateDidLoad : {
        value: function() {
-            this.needsDraw = true;
+            var event = this.treeView.activationEvent;
+            ///// Re-set the activation event
+            if(event && event !== this.activationEvent) {
+                this.delegateEventMap[event] = this.delegateEventMap[this.activationEvent];
+                delete this.delegateEventMap[this.activationEvent];
+                this.activationEvent = this.treeView.activationEvent;
+            }
        }
    },
    prepareForDraw: {
        value : function() {
-            this.activationEvent = this.activationEvent || 'click';
+            var el = this.label._element;
-            this.label._element.addEventListener(this.activationEvent, this.handleNodeActivation.bind(this), false);
+            Object.keys(this.delegateEventMap).forEach(function(event) {
+                el.addEventListener(event, this, false);
+            }, this);
        }
    },
-    handleNodeActivation: {
+    handleEvent : {
        value: function(e) {
-            console.log(this.sourceObject);
+            var delegateMethod = this.delegateEventMap[e._event.type];
-            this.treeView.contentController.delegate.applyPresetSelection(this.sourceObject);
+            this.callDelegateMethod(delegateMethod);
+        }
+    },
+    callDelegateMethod : {
+        value: function(methodName) {
+            var delegate = this.treeView.contentController.delegate;
+            if(delegate && typeof delegate[methodName] === 'function') {
+                delegate[methodName](this.sourceObject);
+            }
        }
    },
    draw : {
@@ -35,6 +53,16 @@ exports.Leaf = Montage.create(TreeNode, {
                this._labelText = this.sourceObject[this.labelKey];
            }
        }
+    },
+    activationEvent : {
+        value : 'click'
+    },
+    delegateEventMap : {
+        value: {
+            'click'     : 'handleNodeActivation',
+            'dragstart' : 'handleDragStart',
+            'dragend'   : 'handleDragEnd'
+        }
    }
author	Valerio Virgillito	2012-02-13 15:08:39 -0800
committer	Valerio Virgillito	2012-02-13 15:08:39 -0800
commit	24b721d49dd20a8ae24198a7dd61dce8838083d9 (patch)
tree	427fc0008ced60bff67b41a165164da491777d97 /js
parent	7b0910720a6979bf41c216c5072d5c7a91e6d0b7 (diff)
parent	85afed8ed707ce20cd28c3139680dfef6fd4b23d (diff)
download	ninja-24b721d49dd20a8ae24198a7dd61dce8838083d9.tar.gz

diff --git a/js/components/treeview/ninja-branch.reel/ninja-branch.html b/js/components/treeview/ninja-branch.reel/ninja-branch.html index 4d557a94..13f1c575 100644 --- a/js/components/treeview/ninja-branch.reel/ninja-branch.html +++ b/js/components/treeview/ninja-branch.reel/ninja-branch.html
@@ -110,7 +110,8 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
110	"name": "Repetition",	110	"name": "Repetition",
111	"properties": {	111	"properties": {
112	"element": { "#": "branchList" },	112	"element": { "#": "branchList" },
113	"contentController": {"@": "arrayController" }	113	"contentController": {"@": "arrayController" },
		114	"selectionEnabled": true
114	}	115	}
115	},	116	},
116		117


diff --git a/js/components/treeview/ninja-branch.reel/ninja-branch.js b/js/components/treeview/ninja-branch.reel/ninja-branch.js index 6b9ebb10..0d1a1d0e 100644 --- a/js/components/treeview/ninja-branch.reel/ninja-branch.js +++ b/js/components/treeview/ninja-branch.reel/ninja-branch.js
@@ -5,7 +5,6 @@
5	</copyright> */	5	</copyright> */
6		6
7	var Montage = require("montage").Montage,	7	var Montage = require("montage").Montage,
8	Component = require("montage/ui/component").Component,
9	TreeNode = require("js/components/treeview/tree-node").TreeNode;	8	TreeNode = require("js/components/treeview/tree-node").TreeNode;
10		9
11	var Branch = exports.Branch = Montage.create(TreeNode, {	10	var Branch = exports.Branch = Montage.create(TreeNode, {


diff --git a/js/components/treeview/ninja-leaf.reel/ninja-leaf.html b/js/components/treeview/ninja-leaf.reel/ninja-leaf.html index bd8e8acd..310274bc 100644 --- a/js/components/treeview/ninja-leaf.reel/ninja-leaf.html +++ b/js/components/treeview/ninja-leaf.reel/ninja-leaf.html
@@ -37,7 +37,7 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
37	</head>	37	</head>
38	<body>	38	<body>
39	<div id="leaf" class="leaf">	39	<div id="leaf" class="leaf">
40	<div id="label" class="leaf-label"></div>	40	<div id="label" class="leaf-label" draggable="true"></div>
41	</div>	41	</div>
42	</body>	42	</body>
43	</html> \ No newline at end of file	43	</html> \ No newline at end of file


diff --git a/js/components/treeview/ninja-leaf.reel/ninja-leaf.js b/js/components/treeview/ninja-leaf.reel/ninja-leaf.js index bd566b26..0b7a171e 100644 --- a/js/components/treeview/ninja-leaf.reel/ninja-leaf.js +++ b/js/components/treeview/ninja-leaf.reel/ninja-leaf.js
@@ -5,7 +5,6 @@ No rights, expressed or implied, whatsoever to this software are provided by Mot
5	</copyright> */	5	</copyright> */
6		6
7	var Montage = require("montage/core/core").Montage,	7	var Montage = require("montage/core/core").Montage,
8	Component = require("montage/ui/component").Component;
9	TreeNode = require("js/components/treeview/tree-node").TreeNode;	8	TreeNode = require("js/components/treeview/tree-node").TreeNode;
10		9
11	exports.Leaf = Montage.create(TreeNode, {	10	exports.Leaf = Montage.create(TreeNode, {
@@ -14,19 +13,38 @@ exports.Leaf = Montage.create(TreeNode, {
14	},	13	},
15	templateDidLoad : {	14	templateDidLoad : {
16	value: function() {	15	value: function() {
17	this.needsDraw = true;	16	var event = this.treeView.activationEvent;
		17
		18	///// Re-set the activation event
		19	if(event && event !== this.activationEvent) {
		20	this.delegateEventMap[event] = this.delegateEventMap[this.activationEvent];
		21	delete this.delegateEventMap[this.activationEvent];
		22	this.activationEvent = this.treeView.activationEvent;
		23	}
18	}	24	}
19	},	25	},
20	prepareForDraw: {	26	prepareForDraw: {
21	value : function() {	27	value : function() {
22	this.activationEvent = this.activationEvent \|\| 'click';	28	var el = this.label._element;
23	this.label._element.addEventListener(this.activationEvent, this.handleNodeActivation.bind(this), false);	29
		30	Object.keys(this.delegateEventMap).forEach(function(event) {
		31	el.addEventListener(event, this, false);
		32	}, this);
		33
24	}	34	}
25	},	35	},
26	handleNodeActivation: {	36	handleEvent : {
27	value: function(e) {	37	value: function(e) {
28	console.log(this.sourceObject);	38	var delegateMethod = this.delegateEventMap[e._event.type];
29	this.treeView.contentController.delegate.applyPresetSelection(this.sourceObject);	39	this.callDelegateMethod(delegateMethod);
		40	}
		41	},
		42	callDelegateMethod : {
		43	value: function(methodName) {
		44	var delegate = this.treeView.contentController.delegate;
		45	if(delegate && typeof delegate[methodName] === 'function') {
		46	delegate[methodName](this.sourceObject);
		47	}
30	}	48	}
31	},	49	},
32	draw : {	50	draw : {
@@ -35,6 +53,16 @@ exports.Leaf = Montage.create(TreeNode, {
35	this._labelText = this.sourceObject[this.labelKey];	53	this._labelText = this.sourceObject[this.labelKey];
36	}	54	}
37	}	55	}
		56	},
		57	activationEvent : {
		58	value : 'click'
		59	},
		60	delegateEventMap : {
		61	value: {
		62	'click' : 'handleNodeActivation',
		63	'dragstart' : 'handleDragStart',
		64	'dragend' : 'handleDragEnd'
		65	}
38	}	66	}
39		67
40		68