Change <header> to be entire top bar

This is more semantically correct.

https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Reference/Roles/banner_role
This commit is contained in:
2025-08-09 21:10:57 -05:00
parent 349672bbad
commit 67de113bdf
2 changed files with 6 additions and 6 deletions

View File

@@ -340,7 +340,7 @@ footer p {
* or it will need its hover background to be chopped off on the left */
}
.header {
.brand {
flex-grow: 1;
margin: 8px var(--page-margin);
font-size: 1.25em;
@@ -348,7 +348,7 @@ footer p {
align-items: center;
}
.header a {
.brand a {
color: inherit;
font-weight: bold;
}

View File

@@ -1,9 +1,9 @@
<div class="top">
<header class="header">
<header class="top">
<div class="brand">
<a href="{{ .Site.Home.Permalink | absLangURL }}">
{{ .Site.Title | markdownify }}
</a>
</header>
</div>
<nav class="navbar" aria-label="{{ i18n "aria_navbar" }}">
{{ with .Site.Home }}
@@ -21,4 +21,4 @@
href="{{ .URL }}">{{ .Name }}</a>
{{ end }}
</nav>
</div>
</header>