loleaflet/src/control/Control.AlertDialog.js | 12 ------------ loleaflet/src/control/Control.ContextMenu.js | 2 +- loleaflet/src/control/Toolbar.js | 15 ++++++++++++++- loleaflet/src/unocommands.js | 1 + 4 files changed, 16 insertions(+), 14 deletions(-)
New commits: commit 0c84261a2a915d2b1e1601d0b77646033c8f2da7 Author: gokaysatir <gokaysa...@collabora.com> AuthorDate: Sat Aug 8 20:32:27 2020 +0300 Commit: Jan Holesovsky <ke...@collabora.com> CommitDate: Tue Aug 11 19:12:24 2020 +0200 loleaflet: remove edit button from open hyperlink modal. Change-Id: I597281b136a41c49954c66f50fd546a209bba75a Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100534 Reviewed-by: Szymon Kłos <szymon.k...@collabora.com> Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> diff --git a/loleaflet/src/control/Control.AlertDialog.js b/loleaflet/src/control/Control.AlertDialog.js index c73d87d92..67224907a 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 19b899d75..4bf98dc0e 100644 --- a/loleaflet/src/control/Toolbar.js +++ b/loleaflet/src/control/Toolbar.js @@ -156,8 +156,20 @@ 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 = command == '.uno:WordCountDialog'; + var hasOpenedDialog = this.dialog.hasOpenedDialog(); if (hasOpenedDialog) { $('.lokdialog_container').addClass('lokblink'); @@ -165,7 +177,8 @@ L.Map.include({ $('.lokdialog_container').removeClass('lokblink'); }, 600); } else 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 449880c74..50434e965 100644 --- a/loleaflet/src/unocommands.js +++ b/loleaflet/src/unocommands.js @@ -221,6 +221,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