Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Navigation Tree View Examples: Add support for aria-selected for indicating currently displayed page #366

Closed
mcking65 opened this issue Apr 20, 2017 · 1 comment
Assignees
Labels
enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern

Comments

@mcking65
Copy link
Contributor

In a navigation tree on a realistic site, the navigation tree exists on all pages included in the tree. So, when the new content or page is displayed, aria-selected indicates which node in the tree represents the current page and that node is in the tab sequence.

In addition, all nodes, including parent nodes, represent pages in the site. So, pressing enter or space on a parent node should activate that node, e.g., display a page associated with that node.

In order to simulate this, instead of opening the target page, click, enter, and space should make a change to a section of the page that displays to the right of the tree. In this section, it could just say something like "Pretend this is the place where the content of the page about "Oranges" or "apples" or whatever is displayed.

Of course, we should keep the wikipedia target URLs in place to demonstrate correct behaviors for opening a new tab or window.

@mcking65 mcking65 added Example Page Related to a page containing an example implementation of a pattern enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy labels Apr 20, 2017
@mcking65 mcking65 added this to the 1.1 Rec milestone Apr 20, 2017
@mcking65 mcking65 removed this from the 1.1 APG Release 4 milestone Aug 13, 2019
@mcking65
Copy link
Contributor Author

This issue is essentially duplicate of #1526. PR #1558 added aria-current to indicate the displayed page when it resolved #1526.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern
Projects
None yet
Development

No branches or pull requests

2 participants