🩹 layouts(navigation): Remove hyperlink on site logo (#167)

Fixes #157.

* removed hyperlink for unicef logo

* 🐛 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

@@ -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>