Skip to content
Snippets Groups Projects

Consolidate user avatar Vue logic

Merged username-removed-408230 requested to merge user-avatar-vue-ce into master
All threads resolved!
1 file
+ 11
11
Compare changes
  • Side-by-side
  • Inline
@@ -3,6 +3,7 @@
import Vue from 'vue';
import collapseIcon from '../icons/collapse_icon.svg';
import UserAvatarImage from '../vue_shared/components/user_avatar/user_avatar_image';
const DiffNoteAvatars = Vue.extend({
props: ['discussionId'],
@@ -15,22 +16,21 @@ const DiffNoteAvatars = Vue.extend({
collapseIcon,
};
},
components: {
'user-avatar-image': UserAvatarImage,
},
template: `
<div class="diff-comment-avatar-holders"
v-show="notesCount !== 0">
<div v-if="!isVisible">
<img v-for="note in notesSubset"
class="avatar diff-comment-avatar has-tooltip js-diff-comment-avatar"
width="19"
height="19"
role="button"
data-container="body"
data-placement="top"
data-html="true"
<user-avatar-image v-for="note in notesSubset"
class="diff-comment-avatar js-diff-comment-avatar"
@click="clickedAvatar($event)
:img-src="note.authorAvatar"
:img-size="19"
:tooltip-text="note.authorName + ': ' + note.noteTruncated"
:data-line-type="lineType"
:title="note.authorName + ': ' + note.noteTruncated"
:src="note.authorAvatar"
@click="clickedAvatar($event)" />
data-html="true"/>
<span v-if="notesCount > shownAvatars"
class="diff-comments-more-count has-tooltip js-diff-comment-avatar"
data-container="body"
Loading