Tree Components - Change the way delegate methods are called

author: Eric Guzman 2012-02-13 10:28:44 -0800
committer: Eric Guzman 2012-02-13 10:28:44 -0800
commit: ef032412216d437ce1c7dfc9050ab41adf0594c0 (patch)
tree: f5cba582ce854907177e620d5529c9e45d945a51 /js
parent: 3a8dc4195a166478695bb286493f83258a4be49b (diff)
download: ninja-ef032412216d437ce1c7dfc9050ab41adf0594c0.tar.gz
4 files changed, 39 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..749df3cd 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,39 @@ 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) {
+            console.log("Delegate method name: ", methodName);
+            var delegate = this.treeView.contentController.delegate;
+            if(delegate && typeof delegate[methodName] === 'function') {
+                delegate[methodName](this.sourceObject);
+            }
        }
    },
    draw : {
@@ -35,6 +54,16 @@ exports.Leaf = Montage.create(TreeNode, {
                this._labelText = this.sourceObject[this.labelKey];
            }
        }
+    },
+    activationEvent : {
+        value : 'click'
+    },
+    delegateEventMap : {
+        value: {
+            'click'     : 'handleNodeActivation',
+            'dragstart' : 'handleDragStart',
+            'dragend'   : 'handleDragEnd'
+        }
    }
author	Eric Guzman	2012-02-13 10:28:44 -0800
committer	Eric Guzman	2012-02-13 10:28:44 -0800
commit	ef032412216d437ce1c7dfc9050ab41adf0594c0 (patch)
tree	f5cba582ce854907177e620d5529c9e45d945a51 /js
parent	3a8dc4195a166478695bb286493f83258a4be49b (diff)
download	ninja-ef032412216d437ce1c7dfc9050ab41adf0594c0.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..749df3cd 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,39 @@ 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	console.log("Delegate method name: ", methodName);
		45	var delegate = this.treeView.contentController.delegate;
		46	if(delegate && typeof delegate[methodName] === 'function') {
		47	delegate[methodName](this.sourceObject);
		48	}
30	}	49	}
31	},	50	},
32	draw : {	51	draw : {
@@ -35,6 +54,16 @@ exports.Leaf = Montage.create(TreeNode, {
35	this._labelText = this.sourceObject[this.labelKey];	54	this._labelText = this.sourceObject[this.labelKey];
36	}	55	}
37	}	56	}
		57	},
		58	activationEvent : {
		59	value : 'click'
		60	},
		61	delegateEventMap : {
		62	value: {
		63	'click' : 'handleNodeActivation',
		64	'dragstart' : 'handleDragStart',
		65	'dragend' : 'handleDragEnd'
		66	}
38	}	67	}
39		68
40		69