From e15b57989b37c00ab1083752a22a402770ea30e8 Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Tue, 3 Sep 2019 16:40:09 +0200
Subject: [PATCH] Fix UI issues

---
 customize.dist/src/less2/include/alertify.less | 9 +++++++++
 www/common/common-ui-elements.js               | 7 ++++---
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/customize.dist/src/less2/include/alertify.less b/customize.dist/src/less2/include/alertify.less
index c73c27c78..683584461 100644
--- a/customize.dist/src/less2/include/alertify.less
+++ b/customize.dist/src/less2/include/alertify.less
@@ -462,6 +462,10 @@
                 }
             }
             margin-bottom: 15px;
+            &:empty {
+                margin: 0;
+                display: none;
+            }
         }
         .cp-share-friend {
             width: 70px;
@@ -493,6 +497,11 @@
                 visibility: hidden;
             }
         }
+        .cp-ownership {
+            & > label {
+                font-weight: bold;
+            }
+        }
     }
 }
 
diff --git a/www/common/common-ui-elements.js b/www/common/common-ui-elements.js
index d73977572..ac855f5c5 100644
--- a/www/common/common-ui-elements.js
+++ b/www/common/common-ui-elements.js
@@ -103,8 +103,8 @@ define([
 
         var redrawAll = function () {};
 
-        var div1 = h('div.cp-share-friends.cp-share-column');
-        var div2 = h('div.cp-share-friends.cp-share-column');
+        var div1 = h('div.cp-share-friends.cp-share-column.cp-ownership');
+        var div2 = h('div.cp-share-friends.cp-share-column.cp-ownership');
         var $div1 = $(div1);
         var $div2 = $(div2);
 
@@ -139,7 +139,6 @@ define([
                 friends: _owners,
                 noFilter: true
             }, function () {
-                console.log(arguments);
             });
             var $div = $(removeCol.div);
             var others1 = removeCol.others;
@@ -162,6 +161,7 @@ define([
                 // Check selection
                 var $sel = $div.find('.cp-share-friend.cp-selected');
                 var sel = $sel.toArray();
+                if (!sel.length) { return; }
                 var me = false;
                 var toRemove = sel.map(function (el) {
                     var ed = $(el).attr('data-ed');
@@ -256,6 +256,7 @@ define([
             $(addButton).click(function () {
                 // Check selection
                 var $sel = $div2.find('.cp-share-friend.cp-selected');
+                if (!sel.length) { return; }
                 var sel = $sel.toArray();
                 var toAdd = sel.map(function (el) {
                     return friends[$(el).attr('data-curve')].edPublic;