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

Jump to bottom

list size is incorrectly calculated #836

Closed
nvaccessAuto opened this issue Aug 19, 2010 · 3 comments
Closed

list size is incorrectly calculated #836

nvaccessAuto opened this issue Aug 19, 2010 · 3 comments
Labels
bug close/duplicate feature/browse-mode

Comments

@nvaccessAuto
Copy link

nvaccessAuto commented Aug 19, 2010 •
edited by jcsteh

Reported by simekk on 2010-08-19 20:17
Using leatest snapshot of NVDA and Firefox 3.6.8

This code:

<ul class="nav">
  <li><a href="index.php">Overview</a></li>
  <li><a href="tabpanel1.php">Example 1</a></li>
  <li><a href="tabpanel2.php">Example 2</a></li>
</ul>

produces:
list with 5 items
visited link Overview
visited link Example 1
visited link Example 2

code is from:
http://test.cita.illinois.edu/aria/tabpanel/tabpanel1.php

Perhaps the blanks are calculated as well but it makes little sense to do so.
@nvaccessAuto
Copy link
Author

Comment 1 by jteh on 2010-08-19 20:44
Yeah, the blanks are being included. I agree this is silly.

To be honest, I'm wondering whether we should just remove the list item count altogether. We don't report list item counts for normal lists, so I don't see why we should for lists on the web, and it is just extra annoying verbosity. If we did this, we'd remove table row/column counts as well.

@dkager
Copy link
Collaborator

dkager commented Jun 9, 2017

The actual code that this issue refers to has been lost, maybe in the transition to GitHub? I'd say close unless the code can be re-added.

@jcsteh
Copy link
Contributor

jcsteh commented Jun 9, 2017

The code was still there, but it was being interpreted as real HTML. I've edited the comment to fix this.

Nevertheless, this is the same issue as #2151, which was fixed in ca1d151. Closinga s duplicate.

@jcsteh jcsteh closed this as completed Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug close/duplicate feature/browse-mode
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dkager @jcsteh @nvaccessAuto