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

NVDA fails to read aria-labelledby for buttons when the attribute refers to the button itself #3551

Closed
nvaccessAuto opened this Issue Sep 25, 2013 · 4 comments

Comments

Projects
None yet
1 participant
@nvaccessAuto

nvaccessAuto commented Sep 25, 2013

Reported by spanchang02 on 2013-09-25 20:44

  1. On tabbing to the button NVDA 2013.2 in FF23 fails to read the aria-labelledby attribute when it refers to its own "value" attribute.
    NVDA reads the value attribute for the button and aria-labelledby does not override it.

Whole Life Policy-1

# The accessible name is the concatenation of the value attribute and h3 text in this case. 1. NVDA fails to read the aria-labelledby for a button when it refers to itself. In this case again the accessible name is the contatenation of the visible button text and the h3 text.

Term Policy-1

Select Policy

Thanks,
Sailesh

Blocked by #1354

@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Sep 25, 2013

Comment 1 by jteh on 2013-09-25 23:10
This only occurs in browse mode. If we rendered the label as specified in browse mode, you would see the heading itself and then the heading content duplicated as part of the button, so I think the current behaviour makes sense. However, it should probably honour the label as specified when tabbing and perhaps quick navigation.

nvaccessAuto commented Sep 25, 2013

Comment 1 by jteh on 2013-09-25 23:10
This only occurs in browse mode. If we rendered the label as specified in browse mode, you would see the heading itself and then the heading content duplicated as part of the button, so I think the current behaviour makes sense. However, it should probably honour the label as specified when tabbing and perhaps quick navigation.

@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Sep 27, 2013

Comment 2 by spanchang02 on 2013-09-27 15:57
Here is a Web page with examples for buttons referred to in the bug report.
Refer to the Add to Cart buttons for Mini Laptop and Back to School Laptop in Section 9 of the page at
http://www.deque.com/techniques-labeling-standard-html-form-controls#tech9

Also, this is not minor: NVDA fails to read the accessible name of the button based on aria-labelledby when one tabs to the button or uses quick nav.
And as NVDA does not announce the nearest heading, the context cannot be discerned by a non sighted user without moving focus away from the element.
Thanks,
Sailesh

nvaccessAuto commented Sep 27, 2013

Comment 2 by spanchang02 on 2013-09-27 15:57
Here is a Web page with examples for buttons referred to in the bug report.
Refer to the Add to Cart buttons for Mini Laptop and Back to School Laptop in Section 9 of the page at
http://www.deque.com/techniques-labeling-standard-html-form-controls#tech9

Also, this is not minor: NVDA fails to read the accessible name of the button based on aria-labelledby when one tabs to the button or uses quick nav.
And as NVDA does not announce the nearest heading, the context cannot be discerned by a non sighted user without moving focus away from the element.
Thanks,
Sailesh

@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Oct 22, 2013

Comment 3 by jteh on 2013-10-22 23:05
Correcting version, since 2013.2 wasn't previously an option. See #3499.

nvaccessAuto commented Oct 22, 2013

Comment 3 by jteh on 2013-10-22 23:05
Correcting version, since 2013.2 wasn't previously an option. See #3499.

@nvaccessAuto

This comment has been minimized.

Show comment
Hide comment
@nvaccessAuto

nvaccessAuto Dec 1, 2014

Comment 4 by jteh on 2014-12-01 06:24
This was fixed by #1354.
Changes:
Added labels: duplicate
State: closed

nvaccessAuto commented Dec 1, 2014

Comment 4 by jteh on 2014-12-01 06:24
This was fixed by #1354.
Changes:
Added labels: duplicate
State: closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment