loleaflet/css/loleaflet.css | 11 +++++------ loleaflet/css/spreadsheet.css | 4 ++-- loleaflet/js/toolbar.js | 2 +- loleaflet/src/layer/AnnotationManager.js | 3 --- loleaflet/src/layer/marker/Annotation.js | 12 ++++++------ loleaflet/src/map/Map.js | 8 ++++---- 6 files changed, 18 insertions(+), 22 deletions(-)
New commits: commit 98492651a65c57d3b628e759a8ad78558208937d Author: Szymon Kłos <szymon.k...@collabora.com> AuthorDate: Wed Jan 30 10:48:32 2019 +0100 Commit: Szymon Kłos <szymon.k...@collabora.com> CommitDate: Wed Jan 30 16:59:06 2019 +0100 Make avatar unified and colored Change-Id: I2991f8d6a5e0660c2ea857e397588f162f557c60 diff --git a/loleaflet/css/loleaflet.css b/loleaflet/css/loleaflet.css index 798612ca7..a6aefc2b2 100644 --- a/loleaflet/css/loleaflet.css +++ b/loleaflet/css/loleaflet.css @@ -302,6 +302,11 @@ body { display: inline-block; } +.avatar-img { + border: solid 2px; + border-radius: 50%; +} + .loleaflet-annotation-author { padding-left: 10px; padding-right: 10px; @@ -354,12 +359,6 @@ body { font-size: 11px; } -.loleaflet-annotation-userline { - background-color: darkblue; - width: 32px; - height: 6px; -} - .loleaflet-annotation-menubar { margin: 0; padding: 0; diff --git a/loleaflet/js/toolbar.js b/loleaflet/js/toolbar.js index b122ad85b..106a1dc71 100644 --- a/loleaflet/js/toolbar.js +++ b/loleaflet/js/toolbar.js @@ -2333,7 +2333,7 @@ function getUserItem(viewId, userName, extraInfo, color) { var html = '<tr class="' + className + '" id="user-' + viewId + '" onclick="onUseritemClicked(event)">' + '<td class=usercolor>'; if (extraInfo !== undefined && extraInfo.avatar !== undefined) { - html += '<img src="' + extraInfo.avatar + '" width="32" height="32" style="border: 5px solid ' + color + ';" />'; + html += '<img class="avatar-img" src="' + extraInfo.avatar + '" style="border-color: ' + color + ';" />'; } // TODO: Add mail and other links as sub-menu. diff --git a/loleaflet/src/layer/AnnotationManager.js b/loleaflet/src/layer/AnnotationManager.js index 80bdd01aa..c8a623639 100644 --- a/loleaflet/src/layer/AnnotationManager.js +++ b/loleaflet/src/layer/AnnotationManager.js @@ -796,15 +796,12 @@ L.AnnotationManager = L.Class.extend({ var initNeeded = (this._initialLayoutData === undefined); var contentWrapperClass = $('.loleaflet-annotation-content-wrapper'); if (initNeeded && contentWrapperClass.length > 0) { - var userlineClass = $('.loleaflet-annotation-userline'); var contentAuthor = $('.loleaflet-annotation-content-author'); var dateClass = $('.loleaflet-annotation-date'); this._initialLayoutData = { wrapperWidth: parseInt(contentWrapperClass.css('width')), wrapperFontSize: parseInt(contentWrapperClass.css('font-size')), - authorLineWidth: parseInt(userlineClass.css('width')), - authorLineHeight: parseInt(userlineClass.css('height')), authorContentHeight: parseInt(contentAuthor.css('height')), dateFontSize: parseInt(dateClass.css('font-size')), }; diff --git a/loleaflet/src/layer/marker/Annotation.js b/loleaflet/src/layer/marker/Annotation.js index 2ea749f94..0eedb26a3 100644 --- a/loleaflet/src/layer/marker/Annotation.js +++ b/loleaflet/src/layer/marker/Annotation.js @@ -193,12 +193,11 @@ L.Annotation = L.Layer.extend({ var tr = L.DomUtil.create('tr', empty, tbody); var tdImg = L.DomUtil.create(tagTd, 'loleaflet-annotation-img', tr); var tdAuthor = L.DomUtil.create(tagTd, 'loleaflet-annotation-author', tr); - var imgAuthor = L.DomUtil.create('img', empty, tdImg); + var imgAuthor = L.DomUtil.create('img', 'avatar-img', tdImg); imgAuthor.setAttribute('src', L.Icon.Default.imagePath + '/user.png'); imgAuthor.setAttribute('width', this.options.imgSize.x); imgAuthor.setAttribute('height', this.options.imgSize.y); this._authorAvatarImg = imgAuthor; - this._authorUserLine = L.DomUtil.create(tagDiv, 'loleaflet-annotation-userline', tdImg); this._contentAuthor = L.DomUtil.create(tagDiv, 'loleaflet-annotation-content-author', tdAuthor); this._contentDate = L.DomUtil.create(tagDiv, 'loleaflet-annotation-date', tdAuthor); @@ -359,6 +358,11 @@ L.Annotation = L.Layer.extend({ $(this._nodeModifyText).text(this._data.text); $(this._contentAuthor).text(this._data.author); $(this._authorAvatarImg).attr('src', this._data.avatar); + var user = this._map.getViewId(this._data.author); + if (user >= 0) { + var color = L.LOUtil.rgbToHex(this._map.getViewColor(user)); + $(this._authorAvatarImg).css('border-color', color); + } var d = new Date(this._data.dateTime.replace(/,.*/, 'Z')); var dateOptions = { weekday: 'short', year: 'numeric', month: 'short', day: 'numeric' }; @@ -385,10 +389,6 @@ L.Annotation = L.Layer.extend({ this._wrapper.style.width = wrapperWidth + 'px'; var wrapperFontSize = Math.round(initialLayoutData.wrapperFontSize * scaleFactor); this._wrapper.style.fontSize = wrapperFontSize + 'px'; - var authorLineWidth = Math.round(initialLayoutData.authorLineWidth * scaleFactor); - this._authorUserLine.style.width = authorLineWidth + 'px'; - var authorLineHeight = Math.round(initialLayoutData.authorLineHeight * scaleFactor); - this._authorUserLine.style.height = authorLineHeight + 'px'; var contentAuthorHeight = Math.round(initialLayoutData.authorContentHeight * scaleFactor); this._contentAuthor.style.height = contentAuthorHeight + 'px'; var dateFontSize = Math.round(initialLayoutData.dateFontSize * scaleFactor); diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js index 06e26f6ca..a2c710eb5 100644 --- a/loleaflet/src/map/Map.js +++ b/loleaflet/src/map/Map.js @@ -271,10 +271,10 @@ L.Map = L.Evented.extend({ updateAvatars: function() { if (this._docLayer && this._docLayer._annotations && this._docLayer._annotations._items) { for (var idxAnno in this._docLayer._annotations._items) { - var username = this._docLayer._annotations._items[idxAnno]._data.author; - if (this._viewInfoByUserName[username]) { - $(this._docLayer._annotations._items[idxAnno]._authorAvatarImg).attr('src', this._viewInfoByUserName[username].userextrainfo.avatar); - } + var annotation = this._docLayer._annotations._items[idxAnno]; + var username = annotation._data.author; + annotation._data.avatar = this._viewInfoByUserName[username].userextrainfo.avatar; + annotation._updateContent(); } } }, commit 9ccc3fdb28b42880b8c75c9b989ee4787552fe38 Author: Szymon Kłos <szymon.k...@collabora.com> AuthorDate: Wed Jan 30 09:31:44 2019 +0100 Commit: Szymon Kłos <szymon.k...@collabora.com> CommitDate: Wed Jan 30 16:58:50 2019 +0100 Reduce readonly toolbar height in calc Change-Id: If1784b0626080792b7f80c57ff956367be390198 diff --git a/loleaflet/css/spreadsheet.css b/loleaflet/css/spreadsheet.css index 8d39c9690..07f27f6e7 100644 --- a/loleaflet/css/spreadsheet.css +++ b/loleaflet/css/spreadsheet.css @@ -6,7 +6,7 @@ } #document-container.spreadsheet-document.readonly { - top: 61px; + top: 56px; } #document-container.spreadsheet-document.tablet { @@ -74,7 +74,7 @@ } #spreadsheet-row-column-frame.readonly { - top: 41px; + top: 36px; } #spreadsheet-row-column-frame.tablet { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits