Changeset 492 for Dev/trunk/src/client/qed-client/pages/question.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/question.js (modified) (2 diffs)

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

@@ -29 +29 @@
         },
         _handlePropertiesChange: function() {
-            if ( this.propertiesForm.validate() ) {
-                lang.mixin(this.object,this.propertiesForm.get('value'));
-            }
+            lang.mixin(this.object,this.propertiesForm.get('value'));
             this.markDirty();
+            this._refresh();
         },
         _handleContentChange: function() {
-            if ( this.contentList.validate() ) {
-                this.object.content = this.contentList.get('value');
-            }
+            this.object.content = this.contentList.get('value');
             this.markDirty();
+            this._refresh();
         },
         _save: function() {
-            if ( this.propertiesForm.validate() && this.contentList.validate() ) {
-                lang.mixin(this.object,this.propertiesForm.get('value'));
-                this.object.content = this.contentList.get('value');
+            if ( this._isValid ) {
                 return this.inherited(arguments);
             } else {
@@ -68 +64 @@
             return false;
         },
+        markDirty: function() {
+            this._isValid = this.propertiesForm.validate() &&
+                            this.contentList.validate();
+            this.saveBtn.set('disabled',!this._isValid);
+            this.saveAndCloseBtn.set('disabled',!this._isValid);
+            this.discardBtn.set('label','Discard & Close');
+            this.inherited(arguments);
+        },
+        markClean: function() {
+            this._isValid = true;
+            this.saveBtn.set('disabled',true);
+            this.saveAndCloseBtn.set('disabled',true);
+            this.discardBtn.set('label','Close');
+            this.inherited(arguments);
+        },
         _ignore: function(evt) {
             if ( evt ) { event.stop( evt ); }