From 6d7274a94ea24343f32fc028984bd3fd5d5f2ee3 Mon Sep 17 00:00:00 2001
From: Maxence Lecanu <lecanu.maxence@gmail.com>
Date: Tue, 25 Jan 2022 09:45:42 +0100
Subject: [PATCH] fix(frontend): add link on logo text

* feat(frontend): add link on logo text

* fix(frontend): display logo-link only if an url is set

Co-authored-by: Maxence Lecanu <maxence@escape.tech>
---
 src/Ombi/ClientApp/src/app/my-nav/my-nav.component.html | 7 ++++++-
 src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss | 9 +++++++++
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.html b/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.html
index 85abdbf0a..f35a8f612 100644
--- a/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.html
+++ b/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.html
@@ -3,7 +3,12 @@
         [attr.role]="(isHandset$ | async) ? 'dialog' : 'navigation'" [mode]="(isHandset$ | async) ? 'over' : 'side'"
         [opened]="!(isHandset$ | async)">
 
-        <mat-toolbar class="application-name" id="nav-applicationName">{{applicationName}}</mat-toolbar>
+        <mat-toolbar class="application-name" id="nav-applicationName">
+            <a *ngIf="applicationUrl; else noApplicationUrlLogo" class="application-name-link" [href]="applicationUrl">{{applicationName}}</a>
+        </mat-toolbar>
+        <ng-template #noApplicationUrlLogo>
+            {{applicationName}}
+        </ng-template>
 
         <mat-nav-list>
             <span  mat-list-item  *ngFor="let nav of navItems">
diff --git a/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss b/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss
index e83ae2d66..80a00c6bb 100644
--- a/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss
+++ b/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss
@@ -165,6 +165,15 @@
         display: flex;
         white-space: normal;
         word-break: break-word;
+
+        .application-name-link {
+            text-decoration: none;
+            color: inherit;
+
+            &:active, &:hover, &:focus {
+                color: inherit;
+            }
+        }
     }
 
     .outer-profile {