Don't always use a link for account avatar

2 files changed, 27 insertions, 19 deletions

diff --git a/client/src/app/shared/shared-account-avatar/account-avatar.component.html b/client/src/app/shared/shared-account-avatar/account-avatar.component.html
index 6bec0b5f3..ca4ceb12f 100644
--- a/client/src/app/shared/shared-account-avatar/account-avatar.component.html
+++ b/client/src/app/shared/shared-account-avatar/account-avatar.component.html
@@ -1,8 +1,15 @@
-<a *ngIf="account" [href]="href" target="_blank" rel="noopener noreferrer" [title]="title">
-  <img
-    [class]="class"
-    [src]="avatarUrl"
-    i18n-alt
-    alt="Account avatar"
-  />
+<ng-template #img>
+  <img [class]="class" [src]="avatarUrl" i18n-alt alt="Account avatar" />
+</ng-template>
+
+<a *ngIf="account && href" [href]="href" target="_blank" rel="noopener noreferrer" [title]="title">
+  <ng-template *ngTemplateOutlet="img"></ng-template>
 </a>
+
+<a *ngIf="account && internalHref" [routerLink]="internalHref" [title]="title">
+  <ng-template *ngTemplateOutlet="img"></ng-template>
+</a>
+
+<ng-container *ngIf="!account || (!href && !internalHref)">
+  <ng-template *ngTemplateOutlet="img"></ng-template>
+</ng-container>
diff --git a/client/src/app/shared/shared-account-avatar/account-avatar.component.ts b/client/src/app/shared/shared-account-avatar/account-avatar.component.ts
index d5533d459..02a0a18bf 100644
--- a/client/src/app/shared/shared-account-avatar/account-avatar.component.ts
+++ b/client/src/app/shared/shared-account-avatar/account-avatar.component.ts
@@ -1,5 +1,4 @@
 import { Component, Input } from '@angular/core'
-import { Account as AccountInterface } from '@shared/models'
 import { Account } from '../shared-main/account/account.model'
 
 @Component({
@@ -8,21 +7,23 @@ import { Account } from '../shared-main/account/account.model'
   templateUrl: './account-avatar.component.html'
 })
 export class AccountAvatarComponent {
-  _href: string
-  _title: string
-
-  @Input() account: { name: string, avatar?: { url?: string }, url: string }
-  @Input() size = '36'
-  @Input() set href (value) {
-    this._href = value
+  @Input() account: {
+    name: string
+    avatar?: { url?: string, path: string }
+    url: string
   }
+  @Input() size: '25' | '34' | '36' | '40' | '120' = '36'
+
+  // Use an external link
+  @Input() href: string
+  // Use routerLink
+  @Input() internalHref: string | string[]
+
   @Input() set title (value) {
     this._title = value
   }
 
-  get href () {
-    return this._href || this.account?.url
-  }
+  private _title: string
 
   get title () {
     return this._title || $localize`${this.account.name} (account page)`
@@ -33,6 +34,6 @@ export class AccountAvatarComponent {
   }
 
   get avatarUrl () {
-    return this.account?.avatar ? this.account.avatar.url : Account.GET_DEFAULT_AVATAR_URL()
+    return Account.GET_ACTOR_AVATAR_URL(this.account)
   }
 }