From bf73c7f6e54ee6b15d59b7a8fbc978877abcc9ec Mon Sep 17 00:00:00 2001
From: Stefan van der Walt <stefanv@berkeley.edu>
Date: Wed, 10 Nov 2021 00:10:20 -0800
Subject: [PATCH] Render language dropdown in theme colors; move to partial

Uses the site parameter "crowdin" (URL) to link to the translation
effort.
---
 assets/css/styles.css              | 12 ++++++++++++
 layouts/partials/navbar-clone.html | 18 +-----------------
 layouts/partials/navbar.html       | 17 +----------------
 layouts/partials/translation.html  | 20 ++++++++++++++++++++
 4 files changed, 34 insertions(+), 33 deletions(-)
 create mode 100644 layouts/partials/translation.html

diff --git a/assets/css/styles.css b/assets/css/styles.css
index 754a6182..cb830a4f 100644
--- a/assets/css/styles.css
+++ b/assets/css/styles.css
@@ -108,6 +108,18 @@ em {
   font-weight: bold;
 }
 
+.navbar.is-fresh .navbar-item.has-dropdown .navbar-dropdown {
+  border-top-color: var(--colorPrimaryDark);
+}
+
+.navbar.is-fresh .navbar-item.has-dropdown:hover .navbar-link {
+  color: var(--colorPrimaryDark);
+}
+
+.navbar.is-fresh .navbar-item.has-dropdown:hover .navbar-link:after {
+  border-color: var(--colorPrimaryDark);
+}
+
 #feature-card {
     height: 475px;
 }
diff --git a/layouts/partials/navbar-clone.html b/layouts/partials/navbar-clone.html
index fd7f9a99..bb5714ba 100644
--- a/layouts/partials/navbar-clone.html
+++ b/layouts/partials/navbar-clone.html
@@ -83,23 +83,7 @@
         {{- end }}
 
         {{- if .IsTranslated }}
-            <div class="navbar-item has-dropdown is-hoverable">
-                <a class="navbar-link">{{ .Language.LanguageName }}</a>
-              <div class="navbar-dropdown">
-              {{- range .Translations }}
-
-              <a href="{{ .RelPermalink }}" id="navbar-item" class="navbar-item">
-                {{ .Language.LanguageName }}
-              </a>
-
-              {{- end }}
-
-              <a href="https://crowdin.com/project/numpyorg"
-              id="navbar-item" class="navbar-item" target="_blank">
-                Help us to translate
-              </a>
-              </div>
-            </div>
+        {{ partial "translation.html" . }}
         {{- end }}
       </div>
     </div>
diff --git a/layouts/partials/navbar.html b/layouts/partials/navbar.html
index 0ebdf0a9..5a454ab5 100644
--- a/layouts/partials/navbar.html
+++ b/layouts/partials/navbar.html
@@ -83,22 +83,7 @@
         {{- end }}
 
         {{- if .IsTranslated }}
-        <div class="navbar-item has-dropdown is-hoverable">
-          <a aria-label="Select language" class="navbar-link">{{ .Language.LanguageName }}</a>
-          <div class="navbar-dropdown">
-            {{- range .Translations }}
-
-            <a href="{{ .RelPermalink }}" id="navbar-item" class="navbar-item">
-              {{ .Language.LanguageName }}
-            </a>
-
-            {{- end }}
-            <a href="https://crowdin.com/project/numpyorg"
-               id="navbar-item" class="navbar-item" target="_blank">
-              Help us to translate
-            </a>
-          </div>
-        </div>
+        {{ partial "translation.html" . }}
         {{- end }}
       </div>
     </div>
diff --git a/layouts/partials/translation.html b/layouts/partials/translation.html
new file mode 100644
index 00000000..4ce0d417
--- /dev/null
+++ b/layouts/partials/translation.html
@@ -0,0 +1,20 @@
+{{ $crowdin := .Site.Params.crowdin }}
+
+<div class="navbar-item has-dropdown is-hoverable">
+ <a aria-label="Select language" class="navbar-link">{{ default .Language .Language.LanguageName }}</a>
+ <div class="navbar-dropdown">
+   {{- range .Translations }}
+
+   <a href="{{ .RelPermalink }}" id="navbar-item" class="navbar-item">
+     {{ default .Language .Language.LanguageName }}
+   </a>
+
+   {{- end }}
+   {{ if $crowdin }}
+   <a href="{{ $crowdin }}"
+      id="navbar-item" class="navbar-item" target="_blank">
+     Help us translate
+   </a>
+   {{ end }}
+ </div>
+</div>