Changeset 441 for Dev/trunk/client/qed/pages/question.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.

File:

• Dev/trunk/client/qed/pages/question.js (modified) (2 diffs)

Dev/trunk/client/qed/pages/question.js

@@ -42 +42 @@
         _refresh: function () {
             this.titleNode.innerHTML = Question.DisplayTitle.get(this.question);
-            this._toolkit.set('value',this.question);
-            this._preview.appendItems(Question.Content.get(this.question));
+            this._toolkit.set('value', this.question);
+            this._preview.set('value', Question.Content.get(this.question));
         },
         _onSave: function(evt) {
-            lang.mixin(this.question, this._toolkit.get('value'));
-            Question.Content.set(this.question, this._preview.getItems());
-            store.put(this.question)
-            .then(function() {
-                Router.go('/questions');
-            },function(err){
-                Content.notify(err,'error');
-            });
+            if ( this._preview.validate() ) {
+                lang.mixin(this.question, this._toolkit.get('value'));
+                Question.Content.set(this.question, this._preview.get('value'));
+                store.put(this.question)
+                .then(function() {
+                    Router.go('/questions');
+                },function(err){
+                    Content.notify(err,'error');
+                });
+            }
             if ( evt ) { event.stop( evt ); }
             return false;
@@ -68 +70 @@
 
             this._preview = new QuestionEditorPreview({ 
-            },this.QuestionEditorPreviewNode);
+                name: 'content',
+                delay: 5,
+                region: 'center'
+            });
             this._preview.startup();
-            this._supportingWidgets.push(this._toolkit, this._preview);
+            this.addChild(this._preview);
         }
     });