Skip to content

Commit

Permalink
Changed aria label to resolve redundancy
Browse files Browse the repository at this point in the history
The main navigation had an aria label that said main navigation.
However, because it is a nav element, this results in hearing the word
"navigation" twice with a screen reader. Changed "main navigation" to
"main" to resolve this.

Signed-off-by: Patrick Smyth <[email protected]>
  • Loading branch information
smythp committed Sep 27, 2023
1 parent 62c403a commit 636d3c0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion layouts/partials/header/header.html
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<div class="header-bar"></div>

<header class="navbar navbar-expand-lg navbar-light doks-navbar">
<nav class="container-{{ if .Site.Params.options.fullWidth }}fluid{{ else }}xxl{{ end }} flex-wrap flex-lg-nowrap" aria-label="Main navigation">
<nav class="container-{{ if .Site.Params.options.fullWidth }}fluid{{ else }}xxl{{ end }} flex-wrap flex-lg-nowrap" aria-label="Main">

<img alt="Sigstore logo" class="nav-image" src="{{ .Site.BaseURL }}/sigstore-logo.png" style="margin-right: 0.25em;height: 2em;">
<a class="navbar-brand order-0" href="{{ .Site.BaseURL | relLangURL }}" aria-label="{{ .Site.Params.Title }}">
Expand Down

0 comments on commit 636d3c0

Please sign in to comment.