Changeset 513 for Dev/trunk/src/client/qed-client/widgets/ObjectBox.js

Timestamp:

03/13/14 22:21:55 (11 years ago)

Author:

hendrikvanantwerpen

Message:

• Another shot at getting change events right for _ComplexValueMixin. Our previous approach made it impossible to detect if a change was cause during startup. Now we work purely with widget 'change' events. Children are connected during postCreate, addChild. If you add children otherwise you need to call connectChildsChanges yourself. Also to make sure events are generated, use _setValueInternal if you really need to set the value attribute yourself.
• Removed unused widget.

File:

• Dev/trunk/src/client/qed-client/widgets/ObjectBox.js (modified) (3 diffs)

Dev/trunk/src/client/qed-client/widgets/ObjectBox.js

@@ -2 +2 @@
     "../lib/object",
     "./LineWithActionsWidget",
+    "./_ComplexValueWidget",
     "dijit/_TemplatedMixin",
     "dijit/_WidgetBase",
@@ -8 +9 @@
     "dojo/_base/lang",
     "dojo/text!./templates/ObjectBox.html"
-], function(objectFuns, LineWithActionsWidget, _TemplatedMixin, _WidgetBase, _WidgetsInTemplateMixin, declare, lang, template) {
-    return declare([_WidgetBase, _TemplatedMixin, _WidgetsInTemplateMixin], {
+], function(objectFuns, LineWithActionsWidget, _ComplexValueWidget, _TemplatedMixin, _WidgetBase, _WidgetsInTemplateMixin, declare, lang, template) {
+    return declare([_ComplexValueWidget], {
         baseClass: "rftObjectBox",
         templateString: template,
-        value: null,
         actions: null,
         startup: function() {
@@ -54 +54 @@
         },
         _showInfoBox: function() {},
-        _setValueAttr: function(value) {
-            this.value = value;
-            this._refresh();
-        },
-        _getValueAttr: function(value) {
-            this.value = value;
-        },
-        _refresh: function() {
-            if ( this.value !== null ) {
-                this.iconNode.className = "rftIcon typeIcon rftIcon"+(this.value.type || '');
-                this.line1.set('title', this.value.title || '');
-                this.line2.set('title', this.value.subTitle || '');
-                this.line3.set('title', this.value.lowerTitle || '');
+        onChange: function(value) {
+            if ( value ) {
+                this.iconNode.className = "rftIcon typeIcon rftIcon"+(value.type || '');
+                this.line1.set('title', value.title || '');
+                this.line2.set('title', value.subTitle || '');
+                this.line3.set('title', value.lowerTitle || '');
             }
         }