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

Tooltip - Ensure ARIA roles, states, and properties are valid - (2036570382) #7845

Closed
Tracked by #7851
dqateam opened this issue Sep 19, 2023 · 2 comments
Closed
Tracked by #7851
Labels
0 - new New issues that need assignment. a11y Issues related to Accessibility fixes or improvements. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review.

Comments

@dqateam
Copy link
Collaborator

dqateam commented Sep 19, 2023

Violation:

Ensure ARIA roles, states, and properties are valid

WCAG Reference:

Severity:

5

Media Type:

ARIA


Areas for Remediation:

  • Title: Tooltip - Ensure ARIA roles, states, and properties are valid - (2036570382)
  • Module: Tooltip
    Issue
    There are tooltip elements with unnecessary ARIA attributes.

User Impact
Assistive technologies may be unable to parse this content.

Code Reference

<calcite-tooltip label="Tooltip for test 1" placement="auto" reference-element="tooltip-auto" aria-hidden="true" aria-label="Tooltip for test 1" aria-live="polite" id="calcite-tooltip-3695e51c-0f68-96d7-ef75-1031b0671f2e" role="tooltip" offset-distance="6" offset-skidding="0" calcite-hydrated="" data-placement="right" style="position: absolute; top: 0px; left: 0px; transform: translate(267px, 217px);" calcite-hydrated-hidden="">
 <p>You have accessed the first test showcasing <code>calcite-tooltip</code>.</p>
</calcite-tooltip>

Suggestion
Ensure ARIA roles, states, and properties are valid. Remove aria-label attribute from <calcite-tooltip> elements.

Compliant Code Example

<calcite-tooltip label="Tooltip for test 1" placement="auto" reference-element="tooltip-auto" aria-hidden="true" aria-live="polite" id="calcite-tooltip-3695e51c-0f68-96d7-ef75-1031b0671f2e" role="tooltip" offset-distance="6" offset-skidding="0" calcite-hydrated="" data-placement="right" style="position: absolute; top: 0px; left: 0px; transform: translate(267px, 217px);" calcite-hydrated-hidden="">
 <p>You have accessed the first test showcasing <code>calcite-tooltip</code>.</p>
</calcite-tooltip>

Additional Resources:

  • Report Source: Product accessibility evaluation conducted on the Calcite Design System in August, 2022.
@dqateam dqateam added 0 - new New issues that need assignment. a11y Issues related to Accessibility fixes or improvements. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review. labels Sep 19, 2023
@driskull
Copy link
Member

The label prop is deprecated so not sure this one is valid.

@geospatialem
Copy link
Member

Agreed, this was mitigated in #6329.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - new New issues that need assignment. a11y Issues related to Accessibility fixes or improvements. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review.
Projects
None yet
Development

No branches or pull requests

3 participants