From b22127f10ca10ab2e8ef92053694f76607757e33 Mon Sep 17 00:00:00 2001
From: ansuz <ansuz@transitiontech.ca>
Date: Thu, 20 Jul 2017 15:55:04 +0200
Subject: [PATCH 1/2] add history removal

---
 customize.dist/translations/messages.js |  3 +++
 www/common/common-messaging.js          | 23 +++++++++++++----------
 www/common/pinpad.js                    |  6 +++++-
 www/contacts/main.less                  | 15 ++++++++++++++-
 4 files changed, 35 insertions(+), 12 deletions(-)

diff --git a/customize.dist/translations/messages.js b/customize.dist/translations/messages.js
index f5ad75bf6..5c0b4c25f 100644
--- a/customize.dist/translations/messages.js
+++ b/customize.dist/translations/messages.js
@@ -282,6 +282,9 @@ define(function () {
     out.contacts_info3 = "Double-click their icon to view their profile";
     out.contacts_info4 = "Either participant can clear permanently a chat history";
 
+    out.contacts_confirmRemoveHistory = 'Are you sure you want to permanently remove your chat history? Data cannot be restored';
+    out.contacts_removeHistoryServerError = 'There was an error while removing your chat history. Try again later';
+
     // File manager
 
     out.fm_rootName = "Documents";
diff --git a/www/common/common-messaging.js b/www/common/common-messaging.js
index 976ed706b..4c426cffe 100644
--- a/www/common/common-messaging.js
+++ b/www/common/common-messaging.js
@@ -284,24 +284,27 @@ define([
         var data = getFriend(common, curvePublic);
         var proxy = common.getProxy();
 
-        var $header = $('<div>', {'class': 'header avatar'}).appendTo($container);
-/*
+        var $header = $('<div>', {
+            'class': 'cp-contact-top-bar header',
+        }).appendTo($container);
+
+        var $avatar = $('<div>', {'class': 'avatar'}).appendTo($header);
+
         var $removeHistory = $('<button>', {
             'class': 'remove-history'
         }).text('remove chat history').click(function () {
-            Cryptpad.confirm('are you sure?', function (yes) {
+            Cryptpad.confirm(common.Messages.contacts_confirmRemoveHistory, function (yes) {
                 if (!yes) { return; }
                 Cryptpad.clearOwnedChannel(data.channel, function (e) {
                     if (e) {
                         console.error(e);
-                        Cryptpad.alert("Something went wrong");
+                        Cryptpad.alert(common.Messages.contacts_removeHistoryServerError);
                         return;
                     }
                 });
             });
         });
-        $removeHistory.appendTo($header); //rightCol);
-*/
+        $removeHistory.appendTo($header);
 
         $('<div>', {'class': 'messages'}).appendTo($container);
         var $inputBlock = $('<div>', {'class': 'input'}).appendTo($container);
@@ -363,14 +366,14 @@ define([
         var $rightCol = $('<span>', {'class': 'right-col'});
         $('<span>', {'class': 'name'}).text(data.displayName).appendTo($rightCol);
         if (data.avatar && avatars[data.avatar]) {
-            $header.append(avatars[data.avatar]);
-            $header.append($rightCol);
+            $avatar.append(avatars[data.avatar]);
+            $avatar.append($rightCol);
         } else {
-            common.displayAvatar($header, data.avatar, data.displayName, function ($img) {
+            common.displayAvatar($avatar, data.avatar, data.displayName, function ($img) {
                 if (data.avatar && $img) {
                     avatars[data.avatar] = $img[0].outerHTML;
                 }
-                $header.append($rightCol);
+                $avatar.append($rightCol);
             });
         }
 
diff --git a/www/common/pinpad.js b/www/common/pinpad.js
index 327ceb2cd..ebece2626 100644
--- a/www/common/pinpad.js
+++ b/www/common/pinpad.js
@@ -142,7 +142,11 @@ define([
                 }
                 rpc.send('CLEAR_OWNED_CHANNEL', channel, function (e, response) {
                     if (e) { return cb(e); }
-                    cb(response);
+                    if (response && response.length) {
+                        cb(void 0, response[0]);
+                    } else {
+                        cb();
+                    }
                 });
             };
 
diff --git a/www/contacts/main.less b/www/contacts/main.less
index 6a26013a7..4ef5efbad 100644
--- a/www/contacts/main.less
+++ b/www/contacts/main.less
@@ -66,7 +66,7 @@ body {
     }
 }
 
-#friendList .friend, #messaging .header {
+#friendList .friend, #messaging .avatar {
     .avatar(30px);
     &.avatar {
         display: flex;
@@ -112,6 +112,19 @@ body {
     }
 }
 
+.cp-contact-top-bar {
+    display: flex;
+    justify-content: space-between;
+    align-items: center;
+    .avatar,
+    .right-col {
+         flex:1 1 auto; //; // 1 auto;
+    }
+    .remove-history {
+        // TODO
+    }
+}
+
 #messaging {
     flex: 1;
     height: 100%;

From dd98e2222a18c062d89bf412e8db0c73a7d06fa4 Mon Sep 17 00:00:00 2001
From: ansuz <ansuz@transitiontech.ca>
Date: Thu, 20 Jul 2017 17:04:11 +0200
Subject: [PATCH 2/2] WIP erase chat history button

---
 www/common/common-messaging.js | 8 +++++---
 www/contacts/main.less         | 9 +++++++--
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/www/common/common-messaging.js b/www/common/common-messaging.js
index fd5f55f88..bfc983914 100644
--- a/www/common/common-messaging.js
+++ b/www/common/common-messaging.js
@@ -289,9 +289,11 @@ define([
 
         var $avatar = $('<div>', {'class': 'avatar'}).appendTo($header);
 
-        var $removeHistory = $('<button>', {
-            'class': 'remove-history'
-        }).text('remove chat history').click(function () {
+        var $removeHistory = $('<span>', {
+            'class': 'remove-history fa fa-eraser'
+        })
+        // TODO set title
+        .click(function () {
             Cryptpad.confirm(common.Messages.contacts_confirmRemoveHistory, function (yes) {
                 if (!yes) { return; }
                 Cryptpad.clearOwnedChannel(data.channel, function (e) {
diff --git a/www/contacts/main.less b/www/contacts/main.less
index 4ef5efbad..ac9af7c78 100644
--- a/www/contacts/main.less
+++ b/www/contacts/main.less
@@ -116,12 +116,17 @@ body {
     display: flex;
     justify-content: space-between;
     align-items: center;
+
     .avatar,
     .right-col {
-         flex:1 1 auto; //; // 1 auto;
+         flex:1 1 auto;
     }
     .remove-history {
-        // TODO
+        height: 100%;
+
+        &:hover {
+            background-color: lighten(@bg-color, 20%);
+        }
     }
 }