Merge pull request #171 from devgeniem/TMS-1015

TMS-1015: Header accessibility fixes

Author: eebbi

CHANGELOG.MD

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+## [1.8.10] - 2024-03-18
+
+- TMS-1015: Header accessibility fixes
+
 ## [1.8.9] - 2024-02-01
 
 - TMS-980: Add redipress_include_search to include component contents in search results

partials/shared/header-inner.dust

@@ -11,14 +11,14 @@
 
             {?Header.lang_nav_horizontal}
                 <div class="is-hidden-touch">
-                    {>"ui/menu/language-nav" links=Header.language_nav.links /}
+                    {>"ui/menu/language-nav" links=Header.language_nav.links id="lang-menu" /}
                 </div>
             {/Header.lang_nav_horizontal}
 
             {?Header.lang_nav_horizontal}
                 <div class="is-hidden-desktop">
                     {?Header.language_nav.links_without_current}
-                        {>"ui/menu/language-nav" links=Header.language_nav.links_without_current /}
+                        {>"ui/menu/language-nav" links=Header.language_nav.links_without_current id="lang-menu-mobile" /}
                     {:else}
                         {>"ui/menu/language-nav-dropdown" links=Header.language_nav.links /}
                     {/Header.language_nav.links_without_current}

partials/ui/menu/language-nav.dust

@@ -1,6 +1,6 @@
 {?links.all}
     <nav aria-label="{Strings.s.header.language_navigation|s}">
-        <ul id="lang-menu"
+        <ul id="{id|attr}"
             class="lang-nav--horizontal ml-6-desktop is-flex is-unstyled is-uppercase has-text-weight-semibold is-family-secondary">
             {#links.all}
                 <li class="ml-2-desktop">

style.css

@@ -2,7 +2,7 @@
  * Theme Name:  TMS Theme Tredu
  * Description: Tampere Multisite Tredu Theme
  * Domain Path: /lang
- * Version:     1.8.9
+ * Version:     1.8.10
  * Author:      Geniem
  * Author URI:  https://geniem.fi
  * Text Domain: tms-theme-tredu