Start a new discussion

To start a new discussion please visit the discussions section of the GitHub home page of the project.

Discussions on GitHub

You can also search our old self-hosted forums for any useful information below but please note that posting new content here is not possible any more.

Menu: not linking to current page breaks some functionality

Home Forums Older releases 1.0.x Menu: not linking to current page breaks some functionality

Viewing 2 posts - 1 through 2 (of 2 total)
  • Author
    Posts
  • #3219
    sascha
    Participant

    Hi there,

    as far as I know it is bad practice in a menu to have a link to the current page. So instead of having an a tag there we replace it with a span tag. Unfortunately that seems to break keyboard navigation and some other functionality.

    Is there any plans of allowing span tags or any other tag instead of a tags in the menu? I guess we could use a class like “current-page-item” to identify and target the span instead of an a.

    Would be great to hear some other thoughts on this.

    And any other practices like using a tag and the use href=”javascript:void();” or use href=”#” are not an option as these are bad practices and not to be encouraged.

    #3220
    sascha
    Participant

    Ok, I did some more research and the solution described in the comments in this article seems to be quite a good compromise:

    http://www.heydonworks.com/article/the-accessible-current-page-link-conundrum

    Basically keeping the link for the current page, but pointing it to the main content of the current page, using it as a skip link for screenreaders. That way it is not confusing for AT users to find sometimes 5 links and sometimes 4 links for instance…

    Topic can be closed from my point of view.

Viewing 2 posts - 1 through 2 (of 2 total)
  • The forum ‘1.0.x’ is closed to new topics and replies.