From ca5ffbe2a9059ac1a4d6d437a655f19f479789de Mon Sep 17 00:00:00 2001
From: Neha Deekonda <85057583+Neha9849@users.noreply.github.com>
Date: Fri, 19 Aug 2022 16:57:19 +0530
Subject: [PATCH] :adhesive_bandage: layouts(navigation): Remove hyperlink on
 site logo (#167)

Fixes #157.

* removed hyperlink for unicef logo

* :bug: layouts(navigation): Remove extraneous HTML, simplify CSS classes

This commit removes the added `<p>` tag, which added extra padding and
spacing between the logo and site title. It takes the `navbar-brand`
class and applies it to the whole `<div>` while moving the `text-center`
class into the site title, so the spacing appears correctly.

This also removes competing CSS classes, simplifying debugging later.

Signed-off-by: Justin W. Flory (he/him) [UNICEF Innovation] <jflory@unicef.org>

Signed-off-by: Justin W. Flory (he/him) [UNICEF Innovation] <jflory@unicef.org>
Co-authored-by: Justin W. Flory (he/him) [UNICEF Innovation] <jflory@unicef.org>
---
 layouts/partials/navigation.html | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/layouts/partials/navigation.html b/layouts/partials/navigation.html
index fd452d8..f79f806 100644
--- a/layouts/partials/navigation.html
+++ b/layouts/partials/navigation.html
@@ -5,15 +5,13 @@
       {{ $orgName:= site.Params.brand.parent_org_name }}
       {{ $orgUrl:= site.Params.brand.parent_org_url }}
 
-      <div class="text-center">
-        <a class="navbar-brand" href="{{ if $logo }}{{ $orgUrl }}{{ else }}{{ site.BaseURL | relLangURL }}{{ end }}">
+      <div class="navbar-brand">
         {{ if $logo }}
         <img class="img-fluid d-inline globalLogoImg"
           src="{{ $logo | absURL }}"
           alt="{{ site.Title }}">
-        </a>
         {{ end }}
-        <a class="navbar-brand text-white d-block px-1" href="{{ site.BaseURL | relLangURL }}">{{ site.Title }}</a>
+        <a class="text-center text-white d-block px-1" href="{{ site.BaseURL | relLangURL }}">{{ site.Title }}</a>
       </div>
       
     </div>