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.

Sign up for GitHub

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

Aria-labelledby should precede aria-label attribute #1283

Closed
nvaccessAuto opened this issue Dec 18, 2010 · 7 comments
Closed

Aria-labelledby should precede aria-label attribute #1283

nvaccessAuto opened this issue Dec 18, 2010 · 7 comments
Labels
ARIA bug p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority

Comments

@nvaccessAuto
Copy link

Reported by simekk on 2010-12-18 18:45
As we have it now, aria-label overrides aria-labelledby, but according to the specification, it should be the other way around.
(http://www.w3.org/TR/wai-aria/complete#textalternativecomputation)

EXAMPLE:

OUTPUT:
this should not be used checkbox checked

Tested under IE8 and latest FF.
@dkager
Copy link
Collaborator

dkager commented Jul 2, 2017

Sounds like a good one for the 2017 browse mode improvement project whose name I conveniently forgot.

@bhavyashah
Copy link

bhavyashah commented Jul 2, 2017 via email

@dkager
Copy link
Collaborator

dkager commented Jul 2, 2017

@bhavyashah Yes.

@jcsteh
Copy link
Contributor

jcsteh commented Jul 2, 2017

I don't think this is a candidate for webfix:

  1. For any browser other than IE, this is determined by the browser, not by NVDA. So, the fixes need to be made in the browsers.
  2. IE specific fixes are low priority for us than other browsers.
  3. Having both aria-labelledby and aria-label seems pretty obscure and unlikely, even in the wild. Webfix is really about issues that hurt quite a number of users in the wild, not spec compliance.

Giving this p4, since I honestly don't see this hurting real use cases. If someone can present a use case where this really matters for IE usage, we might consider revising this to p3.

@jcsteh jcsteh added p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority ARIA labels Jul 2, 2017
@Adriani90
Copy link
Collaborator

@simekk The link in the description is not working anymore. Could you please upload a new test case? Thanks!

@fstrr
Copy link

fstrr commented Aug 16, 2022

I put a test case together for this. The issue has been fixed now, so this can be closed.

@seanbudd
Copy link
Member

Great, thanks @fstrr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ARIA bug p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@dkager @jcsteh @seanbudd @nvaccessAuto @bhavyashah @fstrr @Adriani90