Skip to content

fix(aria-allowed-role): Update allowed roles based on ARIA spec updates#3124

Merged
straker merged 3 commits intodevelopfrom
timogasda-fix-allow-aria-roles
Aug 23, 2021
Merged

fix(aria-allowed-role): Update allowed roles based on ARIA spec updates#3124
straker merged 3 commits intodevelopfrom
timogasda-fix-allow-aria-roles

Conversation

@straker
Copy link
Copy Markdown
Contributor

@straker straker commented Aug 18, 2021

Replaces #3117 due to CircleCI problem of not running tests for outside contributors.

Closes issue: #3082

timogasda and others added 3 commits August 12, 2021 11:57
* `<b>` now allows any roles
* `<nav>` now also allows `menu`, `menubar`, `tablist`
* `<svg>` now allows any roles

Based on ARIA spec: https://www.w3.org/TR/html-aria/
@straker straker requested a review from a team as a code owner August 18, 2021 16:13
@straker straker changed the title Timogasda fix allow aria roles fix(aria-allowed-role): Update allowed roles based on ARIA spec updates Aug 18, 2021
@straker straker merged commit 00f6efc into develop Aug 23, 2021
@straker straker deleted the timogasda-fix-allow-aria-roles branch August 23, 2021 15:41
straker added a commit that referenced this pull request Aug 24, 2021
…es (#3124)

* fix(aria-allowed-role): Update allowed roles based on ARIA spec updates

* `<b>` now allows any roles
* `<nav>` now also allows `menu`, `menubar`, `tablist`
* `<svg>` now allows any roles

Based on ARIA spec: https://www.w3.org/TR/html-aria/

* fix(aria-allowed-role): Update progress element to not allow any roles

Co-authored-by: Timo Gasda <gasda@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants