Skip to content

Accessibility bug fixes#2736

Merged
gingi merged 11 commits intomainfrom
shpaster/a11y-bugs
May 31, 2023
Merged

Accessibility bug fixes#2736
gingi merged 11 commits intomainfrom
shpaster/a11y-bugs

Conversation

@gingi
Copy link
Member

@gingi gingi commented May 25, 2023

No description provided.

@gingi gingi force-pushed the shpaster/a11y-bugs branch from a307c79 to 47d6e4b Compare May 26, 2023 13:58
@codecov
Copy link

codecov bot commented May 26, 2023

Codecov Report

Merging #2736 (037f773) into main (e099763) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 037f773 differs from pull request most recent head 3c81060. Consider uploading reports for the commit 3c81060 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2736   +/-   ##
=======================================
  Coverage   66.24%   66.25%           
=======================================
  Files        1203     1203           
  Lines       33509    33518    +9     
  Branches     6116     6171   +55     
=======================================
+ Hits        22199    22208    +9     
  Misses      11174    11174           
  Partials      136      136           
Impacted Files Coverage Δ
.../src/@batch-flask/ui/form/input/input.directive.ts 85.22% <ø> (ø)
...atch-flask/ui/notifications/notification.module.ts 100.00% <ø> (ø)
...src/app/components/job/action/job-action.module.ts 100.00% <ø> (ø)
...c/app/components/job/details/job-details.module.ts 100.00% <ø> (ø)
...sk/ui/duration-picker/duration-picker.component.ts 91.33% <100.00%> (+0.05%) ⬆️
...elect/select-dropdown/select-dropdown.component.ts 92.59% <100.00%> (+0.70%) ⬆️
...top/src/@batch-flask/ui/select/select.component.ts 82.86% <100.00%> (+0.06%) ⬆️

Continue to review full report in Codecov by Sentry.

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

@gingi gingi force-pushed the shpaster/a11y-bugs branch from 40d8082 to 4a78811 Compare May 30, 2023 19:22
gingi added 2 commits May 30, 2023 15:25
Ensures aria-label on combobox includes control label and value.
@gingi gingi force-pushed the shpaster/a11y-bugs branch from 4a78811 to b19102d Compare May 30, 2023 19:25
@gingi gingi marked this pull request as ready for review May 30, 2023 20:14
Because of quirkiness with Narrator, we place an aria-label at the option element with an explicit "selected/unselected" phrase, and set the nested content to be hidden to avoid reading twice.
@gingi gingi force-pushed the shpaster/a11y-bugs branch from 11d568e to 3f484c4 Compare May 30, 2023 20:19
@gingi gingi force-pushed the shpaster/a11y-bugs branch from 3f484c4 to 3f0fd5d Compare May 30, 2023 20:39
gingi added 2 commits May 30, 2023 16:52
Extra always shown as background-on-foreground (no colors); Sets graph to be aria-hidden.
@gingi gingi merged commit 732b86b into main May 31, 2023
@gingi gingi deleted the shpaster/a11y-bugs branch May 31, 2023 02:33
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