Theme switcher focus fix

adamlaki · adamlaki · commit 3c531d97a937 · 2023-05-23T11:39:58.000+02:00
diff --git a/src/_includes/layouts/base.html b/src/_includes/layouts/base.html
@@ -1,7 +1,7 @@
 {% set assetHash = global.random() %}
 
 <!DOCTYPE html>
-<html lang="en" dir="rtl">
+<html lang="en">
     <head>
         <script src="/js/theme-detection.js?{{ assetHash }}"></script>
         {% include "partials/preload.html" %}
diff --git a/src/css/main.css b/src/css/main.css
@@ -1440,6 +1440,9 @@ samp {
   transform: rotate(45deg);
 }
 
+.breadcrumb-list a {
+  text-decoration: none;
+}
 .breadcrumb-list [aria-current=page] {
   overflow: hidden;
   text-overflow: ellipsis;
diff --git a/src/js/theme-switcher.js b/src/js/theme-switcher.js
@@ -13,6 +13,7 @@
     localStorage.setItem('preferred-theme', theme);
     document.documentElement.setAttribute('data-theme-mode', theme === 'system' ? systemMode : theme);
     themeSwitcher.setAttribute('data-theme-mode', theme);
+    themeSwitcher.querySelector(`.theme-switcher__${theme}-mode`).focus();
   });
 
   themeSwitcher.setAttribute('data-theme-mode', preferredTheme);
diff --git a/src/scss/component/_breadcrumb-list.scss b/src/scss/component/_breadcrumb-list.scss
@@ -31,6 +31,10 @@
     }
   }
 
+  a {
+    text-decoration: none;
+  }
+
   [aria-current='page'] {
     @include text-ellipsis(1);
     display: inline-block;

Original file line number	Diff line number	Diff line change
`@@ -1440,6 +1440,9 @@ samp {`
`1440`	`1440`	`transform: rotate(45deg);`
`1441`	`1441`	`}`
`1442`	`1442`
	`1443`	`+.breadcrumb-list a {`
	`1444`	`+ text-decoration: none;`
	`1445`	`+}`
`1443`	`1446`	`.breadcrumb-list [aria-current=page] {`
`1444`	`1447`	`overflow: hidden;`
`1445`	`1448`	`text-overflow: ellipsis;`