Changeset 384 for Dev/branches/rest-dojo-ui/client/rft/ui/OrderedList.js

Timestamp:

07/29/12 00:03:52 (13 years ago)

Author:

hendrikvanantwerpen

Message:

• Cleanup of the differen ListViews?, List & OrderedList?. Based QuestionListView? on these base classes as well. Removed duplicate code.
• Destroy busy indicator of TabbedQuestionBrowser? (because it's placed outside of the widgets DOM tree, this is not done automatically).
• Use dojo/date/stamp module for uniform and parsable date formatting.
• Added docs/ directory for database format documentation etc.

File:

• Dev/branches/rest-dojo-ui/client/rft/ui/OrderedList.js (modified) (5 diffs)

Dev/branches/rest-dojo-ui/client/rft/ui/OrderedList.js

@@ -2 +2 @@
     'dojo/_base/declare',
     'dojo/_base/lang',
-    'dijit/_WidgetBase',
-    'dijit/_TemplatedMixin',
-    'dijit/_WidgetsInTemplateMixin',
-    'dijit/_Container',
+    'dijit/form/Button',
     'rft/ui/List',
     'dojo/text!./templates/OrderedList.html'
@@ -11 +8 @@
         declare,
         lang,
-        _WidgetBase,
-        _TemplatedMixin,
-        _WidgetsInTemplateMixin,
-        _Container,
+        Button,
         List,
         templateString
         ){
-        return declare('rft.ui.OrderedList',[_WidgetBase,_TemplatedMixin,_WidgetsInTemplateMixin,_Container,List],{
+        return declare('rft.ui.OrderedList',[List],{
             templateString: templateString,
             baseClass: 'rftLineListView',
 
             postCreate: function() {
-                if (this.title) {
-                    this.titleNode.innerHTML = this.title;
-                } else {
-                    this.domNode.removeChild(this.titleNode);
-                }
+                this.inherited(arguments);
 
-                this.source = new dojo.dnd.Source(this.sourceNode, {
-                    isSource: true,
-                    accept: ["LineListViewItem"],
-                    horizontal: false,
-                    withHandles: false,
-                    copyOnly: false,
-                    selfCopy: false,
-                    selfAccept: true,
-                    delay: 0,
-                    singular: true,
-                    creator: lang.hitch(this, this.creatorMethod)
-                });
-
-                new dijit.form.Button({
+                new Button({
                     label: "Move up",
                     showLabel: false,
@@ -49 +26 @@
                     'class': "trans",
                     onClick: lang.hitch(this, function() {
-                        this.moveItem("up");
+                        this._moveSelectedItem("up");
                     })
-                }, this.btnListMoveUp);
+                }, this.btnListMoveUp).startup();
 
-                new dijit.form.Button({
+                new Button({
                     label: "Move down",
                     showLabel: false,
@@ -60 +37 @@
                     'class': "trans",
                     onClick: lang.hitch(this, function() {
-                        this.moveItem("down");
+                        this._moveSelectedItem("down");
                     })
-                }, this.btnListMoveDown); 
+                }, this.btnListMoveDown).startup();
             },
 
-            moveItem: function(dir) {
+            _moveSelectedItem: function(dir) {
                 var node = this.source.getSelectedNodes()[0];
                 if (node) {
@@ -92 +69 @@
                 var anchor = this.source.getSelectedNodes()[0];
                 if (anchor) {
-                    this.source.insertNodes(false, [item], false, anchor);
+                    this.source.insertNodes(false,[item], false, anchor);
                 } else {
-                    this.source.insertNodes(false, [item]);
+                    this.appendItem(item);
                 }
-            },
-            getItems: function() {
-                return this.source.getAllNodes()
-                .map(function(node){
-                    return this.source.getItem(node.id).data;
-                },this);
-            },
+            }
+
         });
     });