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
+ 7
6
Compare changes
  • Side-by-side
  • Inline
@@ -32,11 +32,6 @@ export default {
default: '',
},
},
beforeMount() {
if (!this.svg && !this.src) {
this.src = gon.default_avatar_url;
}
},
computed: {
tooltipClass() {
return this.tooltipText ? 'has-tooltip' : '';
@@ -50,13 +45,19 @@ export default {
imgCss() {
return `avatar ${this.tooltipClass} ${this.avatarSizeClass} ${this.css}`;
},
imgSourceWithFallback() {
if (!this.src && !this.svg) {
return gon.default_avatar_url;
}
return this.src;
},
},
template: `
<span class="avatar-image-container">
<img
v-if="!svg"
:class="imgCss"
:src="src"
:src="imgSourceWithFallback"
:width="size"
:height="size"
:alt="alt"
Loading