Changeset 529 for Dev/trunk/src/client/qed-client/pages/surveyRun.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/surveyRun.js (modified) (3 diffs)

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

@@ -47 +47 @@
             .then(lang.hitch(this,function(allResponses){
                 array.forEach(allResponses, function(response){
-                    var actions = {};
+                    var actions = {}, w;
                     if ( !response.publicationDate ) {
                         actions.Delete = {
-                            callback: function(){
-                                responses.remove(response)
-                                .then(function(){
-                                    w.destroy();
-                                });
-                            },
+                            callback: lang.hitch(this,function(){
+                                // We cannot bind _onDeleteResponse
+                                // directly, because of the
+                                // initialization problem with w. We
+                                // need it in the handler, but we need
+                                // to pass the handler as an argument
+                                // on the creation of w.
+                                this._onDeleteResponse(response,w);
+                            }),
                             properties: {
                                 icon: 'Delete',
@@ -62 +65 @@
                         };
                     }
-                    var w = new LineWithActionsWidget({
+                    w = new LineWithActionsWidget({
                         actions: actions
                     });
@@ -70 +73 @@
                 }, this);
             }));
+        },
+        _onDeleteResponse: function(response,w) {
+            if ( !confirm("Are you sure you want to delete this survey response?") ) {
+                return;
+            }
+            responses.remove(response)
+            .then(function(){
+                w.destroy();
+            });
         },
         _handlePropChange: function(e) {