From e611094dfc396179a680c0c41cfe7bfe57dcb89c Mon Sep 17 00:00:00 2001
From: Thais Siqueira <thais.siqueira@gmail.com>
Date: Wed, 18 Jan 2017 17:58:40 -0200
Subject: [#828] Adds tooltips for encrypt and sign labels

We added a tooltip on the "encrypted" and "signed" label
to inform the user what these flags mean.
And we did some refactoring.

With @tayanefernandes
---
 web-ui/app/js/mail_view/ui/mail_view.js | 95 ++++++++++++++++++---------------
 1 file changed, 51 insertions(+), 44 deletions(-)

(limited to 'web-ui/app/js')

diff --git a/web-ui/app/js/mail_view/ui/mail_view.js b/web-ui/app/js/mail_view/ui/mail_view.js
index 4c9dce30..4315226c 100644
--- a/web-ui/app/js/mail_view/ui/mail_view.js
+++ b/web-ui/app/js/mail_view/ui/mail_view.js
@@ -96,72 +96,79 @@ define(
       };
 
       this.checkEncrypted = function(mail) {
-        if(_.isEmpty(mail.security_casing.locks)) {
-          return {
-            cssClass: 'security-status__label--not-encrypted',
-            label: 'not-encrypted'
-          };
-        }
-
-        var statusClass = ['security-status__label--encrypted'];
+        var statusClass = ['security-status__label'];
         var statusLabel;
+        var tooltip;
 
-        var hasAnyEncryptionInfo = _.any(mail.security_casing.locks, function (lock) {
-          return lock.state === 'valid';
-        });
-
-        if(hasAnyEncryptionInfo) {
-          statusLabel = 'encrypted';
+        if(_.isEmpty(mail.security_casing.locks)) {
+          statusClass.push('--not-encrypted');
+          statusLabel = 'not-encrypted';
+          tooltip = 'This message is not encrypted.';
         } else {
-          statusClass.push('--with-error');
-          statusLabel = 'encryption-error';
+          statusClass.push('--encrypted');
+
+          var hasAnyEncryptionInfo = _.any(mail.security_casing.locks, function (lock) {
+            return lock.state === 'valid';
+          });
+
+          if(hasAnyEncryptionInfo) {
+            statusLabel = 'encrypted';
+            tooltip = 'This message was encrypted.';
+          } else {
+            statusClass.push('--with-error');
+            statusLabel = 'encryption-error';
+            tooltip = 'This message had an encryption error.';
+          }
         }
 
         return {
           cssClass: statusClass.join(''),
-          label: statusLabel
+          label: statusLabel,
+          tooltipText: tooltip
         };
       };
 
       this.checkSigned = function(mail) {
-        var statusNotSigned = {
-          cssClass: 'security-status__label--not-signed',
-          label: 'not-signed'
-        };
-
-        if(_.isEmpty(mail.security_casing.imprints)) {
-          return statusNotSigned;
-        }
+        var statusClass = ['security-status__label'];
+        var statusLabel = [];
+        var tooltip;
 
         var hasNoSignatureInformation = _.any(mail.security_casing.imprints, function (imprint) {
           return imprint.state === 'no_signature_information';
         });
 
-        if(hasNoSignatureInformation) {
-          return statusNotSigned;
-        }
-
-        var statusClass = ['security-status__label--signed'];
-        var statusLabel = ['signed'];
-
-        if(_.any(mail.security_casing.imprints, function(imprint) { return imprint.state === 'from_revoked'; })) {
-          statusClass.push('--revoked');
-          statusLabel.push('signature-revoked');
-        }
+        if(_.isEmpty(mail.security_casing.imprints) || hasNoSignatureInformation) {
+          statusClass.push('--not-signed');
+          statusLabel.push('not-signed');
+          tooltip = 'The sender could not be verified.';
+        } else {
+          statusClass.push('--signed');
+          statusLabel.push('signed');
+          tooltip = 'You are communicating with the real sender.';
+
+          if(_.any(mail.security_casing.imprints, function(imprint) { return imprint.state === 'from_revoked'; })) {
+            statusClass.push('--revoked');
+            statusLabel.push('signature-revoked');
+            tooltip = 'The sender could not be verified.';
+          }
 
-        if(_.any(mail.security_casing.imprints, function(imprint) { return imprint.state === 'from_expired'; })) {
-          statusClass.push('--expired');
-          statusLabel.push('signature-expired');
-        }
+          if(_.any(mail.security_casing.imprints, function(imprint) { return imprint.state === 'from_expired'; })) {
+            statusClass.push('--expired');
+            statusLabel.push('signature-expired');
+            tooltip = 'The sender could not be verified.';
+          }
 
-        if(this.isNotTrusted(mail)) {
-          statusClass.push('--not-trusted');
-          statusLabel.push('signature-not-trusted');
+          if(this.isNotTrusted(mail)) {
+            statusClass.push('--not-trusted');
+            statusLabel.push('signature-not-trusted');
+            tooltip = 'The sender could not be verified.';
+          }
         }
 
         return {
           cssClass: statusClass.join(''),
-          label: statusLabel.join(' ')
+          label: statusLabel.join(' '),
+          tooltipText: tooltip
         };
       };
 
-- 
cgit v1.2.3