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

aria-current="false" is read as aria-current="true" #7892

Closed
ShadowBB opened this Issue Jan 4, 2018 · 4 comments

Comments

Projects
None yet
4 participants
@ShadowBB
Copy link

ShadowBB commented Jan 4, 2018

Steps to reproduce:

  1. Make HTML file with the following txt:
    ARIA current.txt

  2. Open the page (tested with Firefox, Chrome and IE)

  3. Navigate to the the third list item ("item false").

  4. Notice the list item being announced as "Current".

Expected behavior:

The third list item is not announced as "current"

Actual behavior:

The third list item is announced as "current"

System configuration:

NVDA version:
2017.4

NVDA Installed or portable:
Installed

Other information:
NA

Windows version:
Windows 10 Home Version 1709 Build 16299.125

Other questions:

Does the issue still occur after restarting your PC?
Yes.

Have you tried any other versions of NVDA?
No.

@bramd

This comment has been minimized.

Copy link
Contributor

bramd commented Jan 5, 2018

I think this might be according to the ARIA spec (see https://www.w3.org/TR/wai-aria-1.1/#aria-current):

The aria-current attribute is an enumerated type. Any value not included in the list of allowed values SHOULD be treated by assistive technologies as if the value true had been provided.

False is an allowed value, but it is the default and indicated by omitting a value or omitting the whole aria-current attribute. Some feedback of the spec authors would be welcome here.

@leonardder

This comment has been minimized.

Copy link
Collaborator

leonardder commented Jan 6, 2018

@bramd

This comment has been minimized.

Copy link
Contributor

bramd commented Jan 6, 2018

@Adriani90

This comment has been minimized.

Copy link
Collaborator

Adriani90 commented Jan 23, 2019

closing as works for me, solved in #7901. I have tested in IE11, Chrome 71 and Firefox 64.0.2. I cannot reproduce it. @ShadowBB please comment on this if it is still occuring and we will reopen it. Thanks for reporting.

@Adriani90 Adriani90 closed this Jan 23, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.