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

For "ARIA attribute must conform to valid attributes" rule unable to see the ARIA attribute value instead some condition is displayed. #1761

Closed
chandana7393 opened this issue Aug 9, 2019 · 3 comments
Assignees
Labels
fix Bug fixes rule metadata Issues in the rule metadata code (lib/rules)
Milestone

Comments

@chandana7393
Copy link

chandana7393 commented Aug 9, 2019

Steps to reproduce:

  1. Open the below-attached Test file.
  2. Launch the Attest extension.
  3. Click on the 'ANALYZE' button.
    Actual:
    Under "We are not sure this is an issue, because:" observed that for ARIA attribute unable to see the value instead some condition is displayed.
    check_if_ref_attest
    Expected:
    ARIA attribute value must be displayed instead of some condition.

CC: @padmavemulapati
Tested Environment:
Test file:

<!DOCTYPE html>
<html lang="en-US">
<head>
<title>Checking ID reference</title>
</head>
<body>
<main>
<h1>Checking ID reference</h1>
<input id="giftCard_0" aria-describedby="giftCard_0Error" aria-invalid="false"/>
<span id="giftCard_0Error" aria-live="polite">[nothing here yet]</span>
</main>
</body>
</html>

Attest - 2.5.1.21421v
Axe - 3.8.1.21421v
Chrome - 76.0.3809.100v
Firefox - 68.0.1v
OS - Windows 10 64 bit.

@WilcoFiers
Copy link
Contributor

Should check that this isn't in other locals as well.

@WilcoFiers WilcoFiers added the rule metadata Issues in the rule metadata code (lib/rules) label Sep 10, 2019
@WilcoFiers WilcoFiers added this to the Axe-core 3.4 milestone Sep 10, 2019
@straker
Copy link
Contributor

straker commented Sep 10, 2019

This was fixed in #1739 and released in 3.3.1. Once the extensions use the latest axe-core version it should be fixed.

@chandana7393
Copy link
Author

chandana7393 commented Oct 16, 2019

Tested, appearing fine.
attest_valid_values
axe_valid_values
CC: @somaalapati
Tested Environment:
Attest - 2.6.1.28822v
Axe- 4.0.0.28822v
Axe-coconut - 4.0.0.28904v
Chrome - 77.0.3865.120v
Firefox - 69.0.3v
OS - Windows 10 64 bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fixes rule metadata Issues in the rule metadata code (lib/rules)
Projects
None yet
Development

No branches or pull requests

3 participants