loleaflet/css/loleaflet.css | 9 +++++++++ loleaflet/src/control/Control.LokDialog.js | 10 ++++++++++ loleaflet/src/control/Toolbar.js | 8 +++++++- loleaflet/src/layer/tile/CalcTileLayer.js | 3 +++ 4 files changed, 29 insertions(+), 1 deletion(-)
New commits: commit d4bec9e6e151a0b6812f2505fb3f287491e60527 Author: Szymon Kłos <szymon.k...@collabora.com> AuthorDate: Wed Aug 12 10:16:54 2020 +0200 Commit: Szymon Kłos <szymon.k...@collabora.com> CommitDate: Tue Sep 1 11:54:40 2020 +0200 Show avatar for newly inserted comments in calc Change-Id: If602901982e8084856ca5361ab256a93e1045e45 Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100581 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Jan Holesovsky <ke...@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/online/+/101819 Tested-by: Jenkins Reviewed-by: Szymon Kłos <szymon.k...@collabora.com> diff --git a/loleaflet/src/layer/tile/CalcTileLayer.js b/loleaflet/src/layer/tile/CalcTileLayer.js index 534c307ab..a178a59da 100644 --- a/loleaflet/src/layer/tile/CalcTileLayer.js +++ b/loleaflet/src/layer/tile/CalcTileLayer.js @@ -327,6 +327,9 @@ L.CalcTileLayer = (L.Browser.mobile ? L.TileLayer : L.CanvasTileLayer).extend({ var obj = JSON.parse(textMsg.substring('comment:'.length + 1)); obj.comment.tab = parseInt(obj.comment.tab); if (obj.comment.action === 'Add') { + if (obj.comment.author in this._map._viewInfoByUserName) { + obj.comment.avatar = this._map._viewInfoByUserName[obj.comment.author].userextrainfo.avatar; + } var cellPos = L.LOUtil.stringToBounds(obj.comment.cellPos); obj.comment.cellPos = this._convertToTileTwipsSheetArea(cellPos); obj.comment.cellPos = L.latLngBounds(this._twipsToLatLng(obj.comment.cellPos.getBottomLeft()), commit bb015fe41e662f7613338cc8bed0d5394c4783a3 Author: Szymon Kłos <szymon.k...@collabora.com> AuthorDate: Tue Aug 11 07:56:26 2020 +0200 Commit: Szymon Kłos <szymon.k...@collabora.com> CommitDate: Tue Sep 1 11:54:25 2020 +0200 Don't allow to open multiple dialogs and blink active dialog to inform user that he needs to close dialog first... Change-Id: I6c9ee662ae2ab935f03ee8fdd0cf7327703e2170 Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100457 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Jan Holesovsky <ke...@collabora.com> Reviewed-on: https://gerrit.libreoffice.org/c/online/+/101820 Tested-by: Jenkins Reviewed-by: Szymon Kłos <szymon.k...@collabora.com> diff --git a/loleaflet/css/loleaflet.css b/loleaflet/css/loleaflet.css index 28161c6c4..887ba4f18 100644 --- a/loleaflet/css/loleaflet.css +++ b/loleaflet/css/loleaflet.css @@ -577,6 +577,15 @@ body { -o-user-select: none; } +.lokblink { + animation: blink 150ms infinite alternate; +} + +@keyframes blink { + from { opacity:1; } + to { opacity:0; } +} + .form-field-frame { border: 1px solid; position: absolute; diff --git a/loleaflet/src/control/Control.LokDialog.js b/loleaflet/src/control/Control.LokDialog.js index 337acfa5d..0d594414d 100644 --- a/loleaflet/src/control/Control.LokDialog.js +++ b/loleaflet/src/control/Control.LokDialog.js @@ -134,6 +134,16 @@ L.Control.LokDialog = L.Control.extend({ _currentDeck: null, // The sidebar. _calcInputBar: null, // The Formula-Bar. + hasOpenedDialog: function() { + var nonDialogEntries = 0; + for (var index in this._dialogs) { + if (this._dialogs[index].isSidebar || this._dialogs[index].isCalcInputBar) + nonDialogEntries++; + } + + return Object.keys(this._dialogs).length > nonDialogEntries; + }, + _docLoaded: function(e) { if (!e.status) { $('.lokdialog_container').remove(); diff --git a/loleaflet/src/control/Toolbar.js b/loleaflet/src/control/Toolbar.js index 2af30b3ed..c62704634 100644 --- a/loleaflet/src/control/Toolbar.js +++ b/loleaflet/src/control/Toolbar.js @@ -178,7 +178,13 @@ L.Map.include({ } } - if (this.isPermissionEdit() || isAllowedInReadOnly) { + var hasOpenedDialog = this.dialog.hasOpenedDialog(); + if (hasOpenedDialog) { + $('.lokdialog_container').addClass('lokblink'); + setTimeout(function () { + $('.lokdialog_container').removeClass('lokblink'); + }, 600); + } else if (this.isPermissionEdit() || isAllowedInReadOnly) { if (!this.messageNeedsToBeRedirected(command)) this._socket.sendMessage('uno ' + command + (json ? ' ' + JSON.stringify(json) : '')); } _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits