Changeset 441 for Dev/trunk/client/qed/model/widgets/questions/ScaleInputConfigWidget.js

Timestamp:

04/29/13 19:35:10 (12 years ago)

Author:

hendrikvanantwerpen

Message:

Big cleanup of the question content.

• Replaced old list implementations with a new one that behaves like a form widget.
• All question content is now in separate widgets, not in the factory itself.
• Added form and widget validation for question editing.

Location:

Dev/trunk/client/qed/model/widgets/questions

Files:

• . (added)
• ScaleInputConfigWidget.js (moved) (moved from Dev/trunk/client/qed/model/widgets/ScaleInputConfigWidget.js) (6 diffs)

Dev/trunk/client/qed/model/widgets/questions/ScaleInputConfigWidget.js

@@ -1 +1 @@
 define([
-    "dijit/_Container",
-    "dijit/_TemplatedMixin",
-    "dijit/_WidgetBase",
-    "dijit/_WidgetsInTemplateMixin",
+    "../../../widgets/ListWidget",
+    "../../../widgets/_ComplexValueWidget",
     "dijit/form/Button",
     "dijit/form/RadioButton",
     "dijit/form/TextBox",
     "dijit/form/ValidationTextBox",
-    "dijit/form/_FormMixin",
     "dojo/_base/array",
     "dojo/_base/declare",
@@ -20 +17 @@
     "dijit/form/NumberTextBox",
     "dijit/form/TextBox"
-], function(_Container, _TemplatedMixin, _WidgetBase, _WidgetsInTemplateMixin, Button, RadioButton, TextBox, ValidationTextBox, _FormMixin, array, declare, event, lang, domAttr, domConstruct, domStyle, rowTemplate, template) {
+], function(ListWidget, _ComplexValueWidget, Button, RadioButton, TextBox, ValidationTextBox, array, declare, event, lang, domAttr, domConstruct, domStyle, rowTemplate, template) {
 
-    var Row = declare([_WidgetBase,_TemplatedMixin,_WidgetsInTemplateMixin,_Container,_FormMixin],{
+    var Row = declare([_ComplexValueWidget],{
         templateString: rowTemplate,
         hasNA: true,
@@ -29 +26 @@
             this.inherited(arguments);
         },
-        onDestroy: function(e) {
-            this.destroyRecursive();
-            event.stop(e);
-            return false;
-        },
+        onDestroy: function(evt) {},
         _setHasNAAttr: function(value) {
             if ( value === true ) {
@@ -43 +36 @@
     });
     
-    return declare([_WidgetBase,_TemplatedMixin,_WidgetsInTemplateMixin,_Container,_FormMixin],{
+    return declare([_ComplexValueWidget],{
         templateString: template,
         baseClass: "qedScaleWidget",
-        value: null,
         _hasNA: false,
         constuctor: function() {
@@ -52 +44 @@
             this.value = {};
         },
-        onAddItem: function(e) {
-            this._addItem();
+        buildRendering: function() {
+            this.inherited(arguments);
+            this.itemsWidget = new ListWidget({
+                name: "items",
+                type: "ScaleItem",
+                delay: 5,
+                skipForm: true,
+                createListElement: lang.hitch(this, "_createRowWidget"),
+                createAvatar: lang.hitch(this, "_createAvatar")
+            }, this.itemsNode);
+        },
+        _createRowWidget: function(id, item) {
+            var widget = new Row({
+                value: item,
+                hasNA: this._hasNA,
+                id: id,
+                onDestroy: lang.hitch(this,function(evt){
+                    this.itemsWidget.removeItem(id);
+                    event.stop(evt);
+                    return false;
+                })
+            });
+            widget.startup();
+            return widget;
+        },
+        _createAvatar: function(id, item) {
+            return domConstruct.create("div", {
+                innerHTML: item.text || "(empty item)"
+            });
+        },
+        onAddNewItem: function(e) {
+            this.itemsWidget.appendItem({});
             event.stop(e);
             return false;
         },
-        _clearItems: function() {
-            domConstruct.empty(this.itemsNode);
-        },
-        _addItem: function(value) {
-            var item = new Row({
-                name: 'items',
-                value: value,
-                hasNA: this._hasNA
-            });
-            item.placeAt(this.itemsNode);
-            return item;
-        },
-        _setValueAttr: function(value) {
-            this.inherited(arguments);
-            this._clearItems();
-            array.forEach(value.items, function(value){
-                this._addItem(value);
-            }, this);
-        },
         _getValueAttr: function(){
             var value = this.inherited(arguments);
-            if ( value.items && !lang.isArray(value.items) ) {
-                value.items = [value.items];
-            }
             value.type = 'ScaleInput';
             return value;
@@ -89 +89 @@
         },
         _updateNA: function(value) {
-            array.forEach(this.getChildren(), function(child) {
+            array.forEach(this.itemsWidget.getChildren(), function(child) {
                 child.set('hasNA', this._hasNA);
             }, this);