Skip to content

AB#459: Added 'aria-selected' tag for clickable elements#2324

Merged
skapur12 merged 6 commits intomasterfrom
t-skapur/accessibility
May 28, 2021
Merged

AB#459: Added 'aria-selected' tag for clickable elements#2324
skapur12 merged 6 commits intomasterfrom
t-skapur/accessibility

Conversation

@skapur12
Copy link
Member

aria-selected

This screenshot displays how the aria-selected tag for the 3ds Max button is set to 'true' (because the user has selected that button), whereas for all other buttons on the page, the tag is set to 'false.'

@codecov
Copy link

codecov bot commented May 26, 2021

Codecov Report

Merging #2324 (d4bb353) into master (c9ccca9) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2324   +/-   ##
=======================================
  Coverage   65.28%   65.28%           
=======================================
  Files         903      903           
  Lines       25756    25756           
  Branches     5114     5114           
=======================================
  Hits        16815    16815           
  Misses       8941     8941           
Impacted Files Coverage Δ
...-flask/ui/buttons/clickable/clickable.component.ts 98.03% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c9ccca9...d4bb353. Read the comment docs.

@skapur12 skapur12 changed the title Added 'aria-selected' tag for clickable elements AB#459: Added 'aria-selected' tag for clickable elements May 26, 2021
@skapur12 skapur12 requested a review from cRui861 May 27, 2021 00:39
cRui861
cRui861 previously approved these changes May 27, 2021
@cRui861
Copy link
Member

cRui861 commented May 27, 2021

Make sure to squash and merge when merging into master

@skapur12 skapur12 merged commit 9331045 into master May 28, 2021
@skapur12 skapur12 deleted the t-skapur/accessibility branch May 28, 2021 17:28
@cRui861
Copy link
Member

cRui861 commented Aug 5, 2021

Included in #2358

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.

2 participants