From 5441a499ba4bd2e66a24a1e89f0e4e97249a25ff Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Mon, 25 May 2020 17:29:13 +0200
Subject: [PATCH] Fix share and burn buttons in drive toolbar

---
 www/common/drive-ui.js |  4 ++++
 www/drive/inner.js     | 12 ++++++++----
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/www/common/drive-ui.js b/www/common/drive-ui.js
index 0ba77dee2..c44838d6f 100644
--- a/www/common/drive-ui.js
+++ b/www/common/drive-ui.js
@@ -3556,9 +3556,13 @@ define([
             if (!readOnlyFolder) {
                 createNewButton(isInRoot, APP.toolbar.$bottomL);
             }
+            /*
+            // The share button is not displayed anymore in the toolbar: users can't know
+            // if they're going to share the current shared folder or the selected pad
             if (sfId) {
                 createShareButton(sfId, APP.toolbar.$bottomL);
             }
+            */
 
             if (APP.mobile()) {
                 var $context = $('<button>', {
diff --git a/www/drive/inner.js b/www/drive/inner.js
index 831192a52..a3caf7e84 100644
--- a/www/drive/inner.js
+++ b/www/drive/inner.js
@@ -251,14 +251,18 @@ define([
 
             // Add a "Burn this drive" button
             if (!APP.loggedIn && !APP.readOnly) {
-                APP.$burnThisDrive = common.createButton(null, true).click(function () {
+                APP.$burnThisDrive = common.createButton(null, true, {
+                    text: '',
+                    name: 'burn-anon-drive',
+                    icon: 'fa-ban',
+                    title: Messages.fm_burnThisDriveButton,
+                    drawer: false
+                }).click(function () {
                     UI.confirm(Messages.fm_burnThisDrive, function (yes) {
                         if (!yes) { return; }
                         common.getSframeChannel().event('EV_BURN_ANON_DRIVE');
                     }, null, true);
-                }).attr('title', Messages.fm_burnThisDriveButton)
-                  .removeClass('fa-question')
-                  .addClass('fa-ban');
+                });
             }
 
             $('body').css('display', '');