Enhance role=navigation

cpuprogramme / cpu-15

Dotclear theme for radio show C.P.U.

2 stars 1 forks source link

Closed dascritch closed 2 years ago

dascritch commented 2 years ago

<nav role="navigation"> :

role="navigation" is implicit
this <nav> should be ID'd , to visibly hide (and not display : none) only on the ID
<ul role="navigation"> should be <nav><ul>

dascritch commented 2 years ago

The recommended method is… ugly https://accessibility.18f.gov/hidden-content/ but I will.

.sr-hide {
  overflow: hidden;
  clip: rect(0 0 0 0);
  height: 1px;
  width: 1px;
  margin: -1px;
}

dascritch commented 2 years ago

#skip-links {
  height: 0px;
  overflow: hidden;
}

#skip-links:focus-within {
  height: auto;

}

/* remove .skip-links a  */