search

nicolas-steenhout / main

For my main site - jekyll and gh pages
MIT License
0 stars 2 forks source link

Add styling to main menu for current page #32

Closed vavroom closed 4 months ago

vavroom commented 4 months ago

Related to adding aria-current to the main nav.

MDN says:

When you have a group of related elements, such as several links in a breadcrumb or steps in a multi-step flow, with one element in the group styled differently from the others to indicate to the sighted user that this is the current element within its group, the aria-current should be used to inform the assistive technology user what has been indicated via styling.

Should we add a visual styling? This isn't a breadcrumb, etc... But? Dunno. What do you think?

digilou commented 4 months ago

Definitely add a visual styling. Was waiting for the code bit.