Skip to content

Fix AB#1003 Voiceover will now read aloud "+ create new file group"#2745

Merged
cRui861 merged 5 commits intomainfrom
rechen/accessibility
Jun 12, 2023
Merged

Fix AB#1003 Voiceover will now read aloud "+ create new file group"#2745
cRui861 merged 5 commits intomainfrom
rechen/accessibility

Conversation

@cRui861
Copy link
Member

@cRui861 cRui861 commented Jun 8, 2023

  • Also update resource group dropdown to use bl-select instead of mat-autocomplete in create new batch account

Copy link
Member

@gingi gingi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some failing unit tests, but other than making them pass, this looks great. Should make Batch Explorer forms more reliable and look better in general.

@cRui861 cRui861 requested a review from wanghoppe as a code owner June 9, 2023 23:21
cRui861 added 4 commits June 9, 2023 16:22
- switched mat-autocomplete to bl-select for batch-account-create.html
- WIP file-group-picker.html
- WIP file-path-navigator.html
- revert file-path-navigator
@cRui861 cRui861 force-pushed the rechen/accessibility branch from 001f2ab to 43b989d Compare June 9, 2023 23:22
@codecov
Copy link

codecov bot commented Jun 9, 2023

Codecov Report

Merging #2745 (119bbec) into main (d8135c1) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head 119bbec differs from pull request most recent head 64a14ae. Consider uploading reports for the commit 64a14ae to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2745      +/-   ##
==========================================
- Coverage   66.27%   66.26%   -0.01%     
==========================================
  Files        1203     1203              
  Lines       33515    33515              
  Branches     6170     6169       -1     
==========================================
- Hits        22211    22210       -1     
- Misses      11168    11169       +1     
  Partials      136      136              
Impacted Files Coverage Δ
...d/file-group-picker/file-group-picker.component.ts 74.68% <0.00%> (ø)
...ared/file-group-picker/file-group-picker.module.ts 100.00% <ø> (ø)

... and 2 files with indirect coverage changes


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 d8135c1...64a14ae. Read the comment docs.

@cRui861 cRui861 merged commit 7f85086 into main Jun 12, 2023
@cRui861 cRui861 deleted the rechen/accessibility branch June 12, 2023 17:32
gingi added a commit that referenced this pull request Oct 27, 2023
Bug introduced in #2745: No handler for a file group that has changed. Also restores the picker option labels.

Backfills container names on certain API calls because they are not returned, but are needed by various UI flows.
gingi added a commit that referenced this pull request Oct 27, 2023
Bug introduced in #2745: No handler for a file group that has changed. Also restores the picker option labels.

Backfills container names on certain API calls because they are not returned, but are needed by various UI flows.
gingi added a commit that referenced this pull request Nov 1, 2023
Bug introduced in #2745: No handler for a file group that has changed. Also restores the picker option labels.

Backfills container names on certain API calls because they are not returned, but are needed by various UI flows.
gingi added a commit that referenced this pull request Nov 9, 2023
Bug introduced in #2745: No handler for a file group that has changed. Also restores the picker option labels.

Backfills container names on certain API calls because they are not returned, but are needed by various UI flows.
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