loleaflet/src/control/Control.AlertDialog.js |   12 ------------
 loleaflet/src/control/Control.ContextMenu.js |    2 +-
 loleaflet/src/control/Toolbar.js             |   16 +++++++++++++++-
 loleaflet/src/unocommands.js                 |    1 +
 4 files changed, 17 insertions(+), 14 deletions(-)

New commits:
commit 5287950d71f3324fb99b66b6d5fa77fb7ae46125
Author:     gokaysatir <gokaysa...@collabora.com>
AuthorDate: Sat Aug 8 20:32:27 2020 +0300
Commit:     Tamás Zolnai <tamas.zol...@collabora.com>
CommitDate: Tue Aug 11 19:16:04 2020 +0200

    loleaflet: remove edit button from open hyperlink modal.
    
    Change-Id: I597281b136a41c49954c66f50fd546a209bba75a
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100392
    Reviewed-by: Szymon Kłos <szymon.k...@collabora.com>
    Tested-by: Jenkins

diff --git a/loleaflet/src/control/Control.AlertDialog.js 
b/loleaflet/src/control/Control.AlertDialog.js
index c7cff3b8b..8118a30f1 100644
--- a/loleaflet/src/control/Control.AlertDialog.js
+++ b/loleaflet/src/control/Control.AlertDialog.js
@@ -68,18 +68,6 @@ L.Control.AlertDialog = L.Control.extend({
                                });
                        }
 
-                       if (this._map.getDocType() != 'presentation') {
-                               buttonsList.push({
-                                       text: _('Edit'),
-                                       type: 'button',
-                                       className: 
'vex-dialog-button-secondary',
-                                       click: function editClick () {
-                                               vex.closeAll();
-                                               e.map.showHyperlinkDialog();
-                                       }
-                               });
-                       }
-
                        vex.dialog.open({
                                message: messageText,
                                showCloseButton: true,
diff --git a/loleaflet/src/control/Control.ContextMenu.js 
b/loleaflet/src/control/Control.ContextMenu.js
index c4449340e..107da8ba0 100644
--- a/loleaflet/src/control/Control.ContextMenu.js
+++ b/loleaflet/src/control/Control.ContextMenu.js
@@ -22,7 +22,7 @@ L.Control.ContextMenu = L.Control.extend({
                         */
                        general: ['Cut', 'Copy', 'Paste', 'Delete',
                                          'NumberingStart', 
'ContinueNumbering', 'IncrementLevel', 'DecrementLevel',
-                                         'OpenHyperlinkOnCursor', 
'CopyHyperlinkLocation', 'RemoveHyperlink',
+                                         'OpenHyperlinkOnCursor', 
'EditHyperlink', 'CopyHyperlinkLocation', 'RemoveHyperlink',
                                          'AnchorMenu', 'SetAnchorToPage', 
'SetAnchorToPara', 'SetAnchorAtChar',
                                          'SetAnchorToChar', 'SetAnchorToFrame',
                                          'WrapMenu', 'WrapOff', 'WrapOn', 
'WrapIdeal', 'WrapLeft', 'WrapRight', 'WrapThrough',
diff --git a/loleaflet/src/control/Toolbar.js b/loleaflet/src/control/Toolbar.js
index b22be1c83..2af30b3ed 100644
--- a/loleaflet/src/control/Toolbar.js
+++ b/loleaflet/src/control/Toolbar.js
@@ -156,17 +156,31 @@ L.Map.include({
                this._socket.sendMessage(msg);
        },
 
+       messageNeedsToBeRedirected: function(command) {
+               if (command === '.uno:EditHyperlink') {
+                       var that = this;
+                       setTimeout(function () { that.showHyperlinkDialog(); }, 
500);
+                       return true;
+               }
+               else {
+                       return false;
+               }
+       },
+
        sendUnoCommand: function (command, json) {
                var isAllowedInReadOnly = false;
                var allowedCommands = ['.uno:Save', '.uno:WordCountDialog', 
'.uno:EditAnnotation', '.uno:InsertAnnotation', '.uno:DeleteAnnotation'];
+
                for (var i in allowedCommands) {
                        if (allowedCommands[i] === command) {
                                isAllowedInReadOnly = true;
                                break;
                        }
                }
+
                if (this.isPermissionEdit() || isAllowedInReadOnly) {
-                       this._socket.sendMessage('uno ' + command + (json ? ' ' 
+ JSON.stringify(json) : ''));
+                       if (!this.messageNeedsToBeRedirected(command))
+                               this._socket.sendMessage('uno ' + command + 
(json ? ' ' + JSON.stringify(json) : ''));
                }
        },
 
diff --git a/loleaflet/src/unocommands.js b/loleaflet/src/unocommands.js
index be4f9f46e..75caedecd 100644
--- a/loleaflet/src/unocommands.js
+++ b/loleaflet/src/unocommands.js
@@ -230,6 +230,7 @@ var unoCommandsArray = {
        ObjectForwardOne:{global:{menu:_('Forward One'),},},
        OnlineAutoFormat:{text:{menu:_('~While Typing'),},},
        OpenHyperlinkOnCursor:{global:{menu:_('Open Hyperlink'),},},
+       EditHyperlink:{global:{menu:_('Edit Hyperlink'),},},
        OutlineBullet:{global:{menu:_('~Bullets and Numbering...'),},},
        OutlineFont:{global:{menu:_('Outline'),},},
        Overline:{global:{menu:_('Overline'),},},
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to