Changeset 529 for Dev/trunk/src/client/qed-client/pages/surveys.js

Timestamp:

03/23/14 22:58:54 (11 years ago)

Author:

hendrikvanantwerpen

Message:

• Added grey icons for better highlighting, synced block and large buttons.
• Introduced extra color for text, now disabled/inactive/hovered -> it's not very clear though.
• Added confirmation on irrevertable actions.
• Added support for query string in our new request implementation.

File:

• Dev/trunk/src/client/qed-client/pages/surveys.js (modified) (4 diffs)

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

@@ -16 +16 @@
             if ( this._started ) { return; }
             this.inherited(arguments);
+            this.draftsTab.set('title','Drafts (<span class="qedLoading"></span>)');
+            this.publishedTab.set('title','Published (<span class="qedLoading"></span>)');
+            this.runsTab.set('title','Runs (<span class="qedLoading"></span>)');
             this.refresh();
         },
@@ -22 +25 @@
         },
         _onPublishSurvey:function(survey){
-            var self = this;
+            if ( !confirm("After publication the survey cannot be modified anymore, are you sure?") ) {
+                return;
+            }
             survey.publicationDate = new Date();
             surveys.save(survey)
-            .then(function(){
-                self.refreshDrafts();
-                self.refreshPublished();
-            },lang.hitch(this,function(err){
+            .then(lang.hitch(this,function(){
+                this.refreshDrafts();
+                this.refreshPublished();
+            }),lang.hitch(this,function(err){
                 this.notify(err.error,'error');
             }));
         },
         _onDeleteSurvey:function(survey){
-            var self = this;
+            if ( !confirm("Are you sure you want to delete this draft survey?") ) {
+                return;
+            }
             surveys.remove(survey)
-            .then(function(){
-                self.refreshDrafts();
-            },lang.hitch(this,function(err){
+            .then(lang.hitch(this,function(){
+                this.refreshDrafts();
+            }),lang.hitch(this,function(err){
                 this.notify(err.error,'error');
             }));
@@ -62 +69 @@
         },
         _onRunDelete: function(surveyRun) {
+            if ( !confirm("Are you sure you want to delete this survey run?") ) {
+                return;
+            }
             surveyRuns.remove(surveyRun)
             .then(lang.hitch(this,function(){
@@ -135 +145 @@
                                 label: 'Run',
                                 tooltip: 'Run survey',
-                                icon: 'Run'
+                                icon: 'Forward'
                             }
                         }]