Missing aria-current

[gabalafou]

We should mark the current page in the nav using aria-current.

I couldn't find an issue that already tracks this.

[gabalafou]

Point of clarification: we currently use aria-current in the breadcrumb nav but I wonder if we should apply it to the sidebar (and top bar) nav too.

[drammock]

SGTM. Assuming that the top nav only gets aria-current when user is actually on a top-level page right? (not when on any page within that top-level section)

[gabalafou]

SGTM. Assuming that the top nav only gets aria-current when user is actually on a top-level page right? (not when on any page within that top-level section)

Yes that's what I had in mind, plus doing some research into what's the best-practice way to indicate the path parts to assistive tech the same way that we apply an underline in the top nav to the area that the user is in and a notch to the section in the sidebar.