Accessibility bug: Hidden Asides headings in sidebar.php about wprig HOT 3 CLOSED

While I do agree that the Asides title isn't going to be relevant at all for anybody project I do like that it is there and will keep the developer thinking to change it to what it will need to be used for. Maybe we can change it 'Asides' to 'Blog Sidebar' to give the user a better understanding when using a screen reader to navigate an unchanged wprig theme, while still leaving that placeholder for any developer to use in the future.

from wprig.

@LukaszJaro I think a good middle ground would be to make it just 'Sidebar' that way it comes non-opiniated but still gives better context then asides. I made a PR for it

from wprig.

Thanks @dthenley for looking into this. I still think it should be removed completely as even adding the hidden screen reader heading is still being opinionated.

In my case, the asides normally include a visible heading so I would not require a separate hidden screen reader heading. Currently asides are announced as role complementary, if you want to specifically label them best practice appears to be using aria-label:

<aside id="sidebar" aria-label="Sponsors">
  <!-- content -->
</aside>

Example of how this would look:

Leaving out the the hidden heading will probably be most helpful, especially since many will not remove it and may end up with duplicate headings resulting in screen reader noise.

Here's reference https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/complementary_role

from wprig.