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