Skip to content

Commit 3127f41

Browse files
authored
Merge pull request #75 from medialesson/Hide-language-switcher-np
Hide language switcher if only one language is available
2 parents 558c6ce + 82f462c commit 3127f41

File tree

2 files changed

+28
-14
lines changed

2 files changed

+28
-14
lines changed

layouts/_default/baseof.html

Lines changed: 21 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -181,20 +181,27 @@
181181
<footer class="section-container">
182182
<section class="section section--short page-footer-section">
183183
<div class="page-footer-section__start">
184-
<!-- Label for assistive technologies only -->
185-
<label for="language-select" class="sr-only">{{ T "language_menu.label" }}</label>
186-
<select id="language-select" class="select" name="language" onchange="location = this.value;">
187-
{{ range $.Site.Home.AllTranslations }}
188-
<option
189-
value="{{ .RelPermalink }}"
190-
{{ if eq .Language $.Language }}
191-
selected
192-
{{ end }}>
193-
{{ or .Language.LanguageName .Language.Lang | upper }}
194-
</option>
195-
{{ end }}
196-
</select>
197-
184+
<!-- Show language selector only if multiple languages are configured -->
185+
{{ if gt (len .Site.Languages) 1 }}
186+
<!-- Label for assistive technologies only -->
187+
<label for="language-select" class="sr-only">{{ T "language_menu.label" }}</label>
188+
<select
189+
id="language-select"
190+
class="select"
191+
name="language"
192+
onchange="location = this.value;"
193+
data-testid="language-select">
194+
{{ range $.Site.Home.AllTranslations }}
195+
<option
196+
value="{{ .RelPermalink }}"
197+
{{ if eq .Language $.Language }}
198+
selected
199+
{{ end }}>
200+
{{ or .Language.LanguageName .Language.Lang | upper }}
201+
</option>
202+
{{ end }}
203+
</select>
204+
{{ end }}
198205
{{- partial "social-links.html" (
199206
dict
200207
"linkedinUrl" .Site.Params.themes.event.socialLinks.linkedinUrl

tests/baseof.spec.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,3 +17,10 @@ test('Should use short event title as a link to home page in the header menu', a
1717

1818
await expect(page.getByRole('link', { name: /Testcon/ })).toHaveCount(2);
1919
});
20+
21+
test('should not render language selector with one language', async ({ page }) => {
22+
await page.goto('/');
23+
24+
const languageSelector = page.locator('[data-testid="language-select"]');
25+
await expect(languageSelector).toHaveCount(0);
26+
});

0 commit comments

Comments
 (0)