Revert "Issue #25 - Implemented shortcuts on UI"

This reverts commit aa66beb0c74ebaa950a083ed991f6e5f50f9c9ac.
This commit broke the functional tests, so we are reverting it
for now, while we fix it

1 files changed, 8 insertions, 7 deletions

diff --git a/web-ui/app/js/mail_view/ui/mail_actions.js b/web-ui/app/js/mail_view/ui/mail_actions.js
index aa90169f..65cd0aaa 100644
--- a/web-ui/app/js/mail_view/ui/mail_actions.js
+++ b/web-ui/app/js/mail_view/ui/mail_actions.js
@@ -37,13 +37,11 @@ define(
         moreActions: '#more-actions'
       });
 
-      this.deleteMail = function () {
-        this.trigger(document, events.ui.mail.delete, {mail: this.attr.mail});
-        this.select('moreActions').hide();
-      };
 
       this.displayMailActions = function () {
+
         this.$node.html(templates.mails.mailActions());
+
         this.select('moreActions').hide();
 
         this.on(this.select('replyButtonTop'), 'click', function () {
@@ -55,6 +53,11 @@ define(
           this.select('moreActions').hide();
         }.bind(this));
 
+        this.on(this.select('deleteButtonTop'), 'click', function () {
+          this.trigger(document, events.ui.mail.delete, {mail: this.attr.mail});
+          this.select('moreActions').hide();
+        }.bind(this));
+
         this.on(this.select('viewMoreActions'), 'click', function () {
           this.select('moreActions').toggle();
         }.bind(this));
@@ -69,14 +72,12 @@ define(
             this.select('moreActions').hide();
           }
         }.bind(this));
+
       };
 
       this.after('initialize', function () {
         this.on(document, events.dispatchers.rightPane.clear, this.teardown);
-        this.on(document, events.shortcuts.deleteMail, this.deleteMail);
-
         this.displayMailActions();
-        this.on(this.select('deleteButtonTop'), 'click', this.deleteMail);
       });
     }
   }