Changeset 492 for Dev/trunk/src/client/qed-client/pages/survey.js

Timestamp:

03/09/14 14:23:42 (11 years ago)

Author:

hendrikvanantwerpen

Message:

• Enable/disable buttons on content change.
• One place to do date formatting, because it was going wrong again.
• Serialize questions in survey properly.
• _ComplexValueMixin consumes submit events, but does trigger outer forms if present.
• Trigger dialog show/hide for login only after previous effect is finished.
• Check that documents are actually valid, not just that validator returned a result.
• Validate email and timestamp formats.
• Prepared for live runs.

File:

• Dev/trunk/src/client/qed-client/pages/survey.js (modified) (5 diffs)

Dev/trunk/src/client/qed-client/pages/survey.js

@@ -23 +23 @@
             this._setupQuestionBrowser();
             this._setupListView();
+            this.questionList.on('change',lang.hitch(this,'_handleQuestionsChange'));
             this._load();
         },
@@ -58 +59 @@
         _refresh: function() {
             this.titleNode.innerHTML = this.object.title || "(set title in properties)";
-            this.propertiesDialog.set('value',{survey:this.object});
+            this.propertiesForm.set('value',{survey:this.object});
             this.questionList.set('value',
                                   this.object.questions);
         },
-        _save: function() {
-            this.object.questions = this.questionList.get('value');
-            return this.inherited(arguments);
-        },
         _includeQuestion: function(question) {
             this.questionList.appendItem(question);
+        },
+        _handleQuestionsChange: function() {
+            this.object.questions = this.questionList.get('value');
+            this.markDirty();
+            this._refresh();
         },
         _onShowProperties: function(evt) {
@@ -76 +78 @@
         _onPropertiesOk: function(evt) {
             this.propertiesDialog.hide();
-            lang.mixin(this.object, this.propertiesDialog.get('value').survey);
+            lang.mixin(this.object, this.propertiesForm.get('value').survey);
             this.markDirty();
             this._refresh();
@@ -84 +86 @@
         _onPropertiesCancel: function(evt) {
             this.propertiesDialog.hide();
-            this.propertiesDialog.set('value',{survey:this.object});
+            this.propertiesForm.set('value',{survey:this.object});
             if ( evt ) { event.stop(evt); }
             return false;
@@ -109 +111 @@
                 preview: true
             });
+        },
+        markDirty: function() {
+            this.saveBtn.set('disabled',false);
+            this.saveAndCloseBtn.set('disabled',false);
+            this.discardBtn.set('label','Discard & Close');
+            this.inherited(arguments);
+        },
+        markClean: function() {
+            this.saveBtn.set('disabled',true);
+            this.saveAndCloseBtn.set('disabled',true);
+            this.discardBtn.set('label','Close');
+            this.inherited(arguments);
         }
     });