From 66b076111e040c3f4b4765ff519fb1e684b2b22e Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Thu, 15 Jun 2017 12:32:44 +0200
Subject: [PATCH] Add an upload button to the slide's file picker

---
 customize.dist/translations/messages.fr.js |  3 ++-
 customize.dist/translations/messages.js    |  3 ++-
 www/common/cryptpad-common.js              | 19 ++++++++++++++++
 www/drive/main.js                          | 25 +++++-----------------
 www/slide/main.js                          | 24 ++++++++-------------
 5 files changed, 37 insertions(+), 37 deletions(-)

diff --git a/customize.dist/translations/messages.fr.js b/customize.dist/translations/messages.fr.js
index e4e6befbd..2887e3748 100644
--- a/customize.dist/translations/messages.fr.js
+++ b/customize.dist/translations/messages.fr.js
@@ -134,8 +134,9 @@ define(function () {
 
     out.filePickerButton = "Intégrer un fichier";
     out.filePicker_close = "Fermer";
-    out.filePicker_description = "Choisissez un fichier de votre CryptDrive pour l'intégrer";
+    out.filePicker_description = "Choisissez un fichier de votre CryptDrive pour l'intégrer ou uploadez-en un nouveau";
     out.filePicker_filter = "Filtrez les fichiers par leur nom";
+    out.or = 'ou';
 
     out.slideOptionsTitle = "Personnaliser la présentation";
     out.slideOptionsButton = "Enregistrer (Entrée)";
diff --git a/customize.dist/translations/messages.js b/customize.dist/translations/messages.js
index 16bc6aff4..cad60ee63 100644
--- a/customize.dist/translations/messages.js
+++ b/customize.dist/translations/messages.js
@@ -136,8 +136,9 @@ define(function () {
 
     out.filePickerButton = "Embed a file";
     out.filePicker_close = "Close";
-    out.filePicker_description = "Choose a file from your CryptDrive to embed it";
+    out.filePicker_description = "Choose a file from your CryptDrive to embed it or upload a new one";
     out.filePicker_filter = "Filter files by name";
+    out.or = 'or';
 
     out.slideOptionsTitle = "Customize your slides";
     out.slideOptionsButton = "Save (enter)";
diff --git a/www/common/cryptpad-common.js b/www/common/cryptpad-common.js
index eec76cc6c..df2f17379 100644
--- a/www/common/cryptpad-common.js
+++ b/www/common/cryptpad-common.js
@@ -945,6 +945,25 @@ define([
                     }));
                 }
                 break;
+            case 'upload':
+                button = $('<button>', {
+                    'class': 'btn btn-primary new',
+                    title: Messages.uploadButtonTitle,
+                }).append($('<span>', {'class':'fa fa-upload'})).append(' '+Messages.uploadButton);
+                if (!data.FM) { return; }
+                var $input = $('<input>', {
+                    'type': 'file',
+                    'style': 'display: none;'
+                }).on('change', function (e) {
+                    var file = e.target.files[0];
+                    var ev = {
+                        target: data.target
+                    };
+                    data.FM.handleFile(file, ev);
+                    if (callback) { callback(); }
+                });
+                button.click(function () { $input.click(); })
+                break;
             case 'template':
                 if (!AppConfig.enableTemplates) { return; }
                 button = $('<button>', {
diff --git a/www/drive/main.js b/www/drive/main.js
index cfa794cbf..82b916a87 100644
--- a/www/drive/main.js
+++ b/www/drive/main.js
@@ -1439,26 +1439,11 @@ define([
         var createUploadButton = function () {
             var inTrash = filesOp.isPathIn(currentPath, [TRASH]);
             if (inTrash) { return; }
-            var $icon = $('<span>', {
-                'class': 'fa fa-upload'
-            });
-            var $input = $('<input>', {
-                'type': 'file',
-                'style': 'display: none;'
-            }).on('change', function (e) {
-                var file = e.target.files[0];
-                var ev = {
-                    target: $content[0]
-                };
-                APP.FM.handleFile(file, ev);
-            });
-            var $button = $('<button>', {
-                'class': 'btn btn-primary new',
-                title: Messages.uploadButtonTitle
-            }).append($icon).append(' '+Messages.uploadButton).click(function () {
-                $input.click();
-            });
-            return $button;
+            var data = {
+                FM: APP.FM,
+                target: $content[0]
+            };
+            return Cryptpad.createButton('upload', false, data);
         };
 
         var hideNewButton = function () {
diff --git a/www/slide/main.js b/www/slide/main.js
index 50efb8b2c..ad6b8652b 100644
--- a/www/slide/main.js
+++ b/www/slide/main.js
@@ -244,6 +244,11 @@ define([
                     if (to) { window.clearTimeout(to); }
                     to = window.setTimeout(updateContainer, 300);
                 });
+                $filter.append(' '+Messages.or+' ');
+                var data = {FM: APP.FM};
+                $filter.append(Cryptpad.createButton('upload', false, data, function () {
+                    $block.hide();
+                }));
                 updateContainer();
                 $body.keydown(function (e) {
                     if (e.which === 27) { $block.hide(); }
@@ -578,27 +583,16 @@ define([
                     dropArea: $iframe.find('.CodeMirror'),
                     body: $iframe.find('body'),
                     onUploaded: function (ev, data) {
-                        console.log(ev, data);
-                        /* 
-                        TODO: test if getCursor() works
-                        If we can drop a file without updating the cursor, we'll need the following
-                        code to get the cursor position from the drop event
-                        var obj = {
-                            left: ev.originalEvent.pageX,
-                            top: ev.originalEvent.pageY,
-                        };
-                        var cursor = editor.coordsChar(obj);
-                        */
                         //var cursor = editor.getCursor();
                         var cleanName = data.name.replace(/[\[\]]/g, '');
-                        var text = '['+cleanName+']('+data.url+')';
-                        if (data.mediatag) {
+                        var text = '!['+cleanName+']('+data.url+')';
+                        /*if (data.mediatag) {
                             text = '!'+text;
-                        }
+                        }*/
                         editor.replaceSelection(text);
                     }
                 };
-                Cryptpad.createFileManager(fmConfig);
+                APP.FM = Cryptpad.createFileManager(fmConfig);
             };
 
             config.onRemote = function () {