diff --git a/apps/files/css/files.scss b/apps/files/css/files.scss index 1fc38f0367..80a95559b2 100644 --- a/apps/files/css/files.scss +++ b/apps/files/css/files.scss @@ -594,8 +594,8 @@ a.action > img { max-width: 70px; overflow: hidden; text-overflow: ellipsis; - vertical-align: bottom; - padding-left: 6px; + vertical-align: middle; + margin-left: 6px; } #fileList .favorite-mark.permanent { diff --git a/core/js/share.js b/core/js/share.js index fd21a25a2d..35214fb11b 100644 --- a/core/js/share.js +++ b/core/js/share.js @@ -203,11 +203,13 @@ OC.Share = _.extend(OC.Share || {}, { * @param {String} remoteAddress full remote share * @return {String} HTML code to display */ - _formatRemoteShare: function(remoteAddress) { + _formatRemoteShare: function(remoteAddress, message) { var parts = this._REMOTE_OWNER_REGEXP.exec(remoteAddress); if (!parts) { - // display as is, most likely to be a simple owner name - return '' + t('core', 'Shared by {recipient}', {recipient: escapeHTML(remoteAddress)}) + ''; + // display avatar of the user + var avatar = ''; + var hidden = '' + message + escapeHTML(remoteAddress) + ''; + return avatar + hidden; } var userName = parts[1]; @@ -243,7 +245,7 @@ OC.Share = _.extend(OC.Share || {}, { _formatShareList: function(recipients) { var _parent = this; return $.map(recipients, function(recipient) { - recipient = _parent._formatRemoteShare(recipient); + recipient = _parent._formatRemoteShare(recipient, t('core', 'Shared with ')); return recipient; }); }, @@ -259,8 +261,7 @@ OC.Share = _.extend(OC.Share || {}, { var action = $tr.find('.fileactions .action[data-action="Share"]'); var type = $tr.data('type'); var icon = action.find('.icon'); - var message; - var recipients; + var message, recipients, avatars; var owner = $tr.attr('data-share-owner'); var shareFolderIcon; var iconClass = 'icon-shared'; @@ -297,12 +298,12 @@ OC.Share = _.extend(OC.Share || {}, { message = t('core', 'Shared'); // even if reshared, only show "Shared by" if (owner) { - message = this._formatRemoteShare(owner); + message = t('core', 'Shared by '); + avatars = this._formatRemoteShare(owner, message); + } else if (recipients) { + avatars = this._formatShareList(recipients.split(", ")).join(""); } - else if (recipients) { - message = t('core', 'Shared with {recipients}', {recipients: this._formatShareList(recipients.split(", ")).join(", ")}, 0, {escape: false}); - } - action.html(' ' + message + '').prepend(icon); + action.html(avatars).prepend(icon); if (owner || recipients) { var avatarElement = action.find('.avatar');