Skip to content

Sub-Task/TUP-573 Add accessibility role(button) to dropdown a-tags#757

Merged
wesleyboar merged 1 commit intotask/tup-595-accessibilityfrom
sub-task/tup-573-add-button-role-to-a-tags
Dec 4, 2023
Merged

Sub-Task/TUP-573 Add accessibility role(button) to dropdown a-tags#757
wesleyboar merged 1 commit intotask/tup-595-accessibilityfrom
sub-task/tup-573-add-button-role-to-a-tags

Conversation

@R-Tomas-Gonzalez
Copy link
Copy Markdown
Contributor

Overview

Adds role='button' to a-tags in dropdown.

Related

TUP-573

Changes

Adds role='button' to a-tags in dropdown.

Testing

  1. Follow https://github.com/TACC/Core-CMS/wiki/Locally-Develop-CMS-and-TUP-CMS to test changes in this on tup-ui
  2. Check for a-tag roles on dropdowns (role='button')

UI

Screenshot 2023-12-01 at 5 22 04 PM Screenshot 2023-12-01 at 4 01 48 PM

@R-Tomas-Gonzalez R-Tomas-Gonzalez added priority ━ Medium priority effort XS Extra small effort task (~¼ a week) — does not include review time labels Dec 1, 2023
@R-Tomas-Gonzalez R-Tomas-Gonzalez self-assigned this Dec 1, 2023
@wesleyboar wesleyboar merged commit c0cff01 into task/tup-595-accessibility Dec 4, 2023
@wesleyboar wesleyboar deleted the sub-task/tup-573-add-button-role-to-a-tags branch December 4, 2023 16:29
wesleyboar added a commit that referenced this pull request Dec 13, 2023
* adding role=button to a-tags (#757)

* sub-task/tup-579 Added navbarDropdown id to a tag (#758)

* fix: tup 662 nav dropdown id not unique (#762)

* fix: tup-662 nav dropdown id

* fix: tup-662 nav dropdown id, simpler solution

* fix: tup-662 nav dropdown id, follow MDN a11y menu role docs

https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/menu_role

---------

Co-authored-by: Tomas Gonzalez <63771558+R-Tomas-Gonzalez@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort XS Extra small effort task (~¼ a week) — does not include review time priority ━ Medium priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants