Skip to content

bug: AXE: Elements must use only allowed ARIA attributes for ion-select #22715

Open

Description

@kensodemann

Bug Report

Ionic version:

[ ] 4.x
[x] 5.x

Current behavior:
Axe issue - Elements must use only allowed ARIA attributes for ionselect.

Running AXE on the app linked below I get the following for ion-select:

Fix the following:
ARIA attribute is not allowed: aria-required="true"

Expected behavior:
Would be able to use aria-required on an ion-select without the AXE error.

Related code:

https://github.com/ionic-afl/ionic-v5-observations.git

Activity

  1. ionitron-bot commented on Dec 23, 2020

    @ionitron-bot

    Thanks for the issue! This issue has been labeled as holiday triage. With the winter holidays quickly approaching, much of the Ionic Team will soon be taking time off. During this time, issue triaging and PR review will be delayed until the team begins to return. After this period, we will work to ensure that all new issues are properly triaged and that new PRs are reviewed.

    In the meantime, please read our Winter Holiday Triage Guide for information on how to ensure that your issue is triaged correctly.

    Thank you!

  2. added this to the a11y milestone on Dec 23, 2020
  3. kensodemann commented on Feb 15, 2021

    @kensodemann
    ContributorAuthor

    Discussing with the user, this also results in them not being able to mark the select required in a way that will be properly picked up by a screen reader.

  4. modified the milestones: a11y, 5.6.0 on Feb 18, 2021
  5. added
    type: buga confirmed bug report
    and removed
    holiday triageissues that were created during holiday period
    on Feb 18, 2021
  6. removed this from the 5.6.0 milestone on Mar 3, 2021
  7. kensodemann commented on Mar 18, 2021

    @kensodemann
    ContributorAuthor

    There are issues with the HTML specification itself. aria-required is not something that is allowed, and the spec for custom elements does not allow for a "required" attribute. Thus there is currently no way within the HTML specs to support this.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions