ampstart icon indicating copy to clipboard operation
ampstart copied to clipboard

[A11Y] add `aria-current` to navigation lists

Open aghassemi opened this issue 8 years ago • 2 comments

On the main ampstart.com, where-ever the CSS www-current-page exists, we also need to add aria-current="page"

for templates, similar whenever a navlink is "highlighted", aria-current="page" should also be added.

aghassemi avatar Nov 17 '17 16:11 aghassemi

(will need to wait for https://github.com/ampproject/amphtml/pull/12102 to be in prod. ETA ~3 weeks)

aghassemi avatar Nov 17 '17 16:11 aghassemi

i just got rid ofwww-current-page in the recent updates. But there could be a way to add the aria. Thanks for filing

camelburrito avatar Nov 17 '17 16:11 camelburrito