Skip to content

Fix aria-allowed-attr errors on config page camera button and video effect items#4662

Merged
JamesBurnside merged 3 commits intomainfrom
jaburnsi/fix-a11y-aria-label-errs
May 27, 2024
Merged

Fix aria-allowed-attr errors on config page camera button and video effect items#4662
JamesBurnside merged 3 commits intomainfrom
jaburnsi/fix-a11y-aria-label-errs

Conversation

@JamesBurnside
Copy link
Copy Markdown
Member

@JamesBurnside JamesBurnside commented May 27, 2024

What

Why

Fixes these a11y errs:

image
image

More info: https://skype.visualstudio.com/SPOOL/_workitems/edit/3673658

How Tested

Accessibility insights tool no longer reports error.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6176192
  • Base size: 6176178
  • Diff size: 14

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2024

Chat bundle size is increased❗.

  • Current size: 2061900
  • Base size: 2061886
  • Diff size: 14

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2024

Calling bundle size is increased❗.

  • Current size: 4851605
  • Base size: 4851591
  • Diff size: 14

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2024

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 26056 / 40680
64.05%
26056 / 40680
64.05%
710 / 1268
55.99%
2077 / 3273
63.45%
Current 26056 / 40680
64.05%
26056 / 40680
64.05%
710 / 1268
55.99%
2077 / 3273
63.45%
Diff 0 / 0
0%
0 / 0
0%
0 / 0
0%
0 / 0
0%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2024

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 50253 / 80860
62.14%
50253 / 80860
62.14%
1028 / 2283
45.02%
2934 / 4819
60.88%
Current 50251 / 80860
62.14%
50251 / 80860
62.14%
1028 / 2283
45.02%
2950 / 4827
61.11%
Diff -2 / 0
0%
-2 / 0
0%
0 / 0
0%
16 / 8
0.23%

@JamesBurnside JamesBurnside enabled auto-merge (squash) May 27, 2024 20:28
@github-actions
Copy link
Copy Markdown
Contributor

@JamesBurnside JamesBurnside merged commit 0bfe50a into main May 27, 2024
@JamesBurnside JamesBurnside deleted the jaburnsi/fix-a11y-aria-label-errs branch May 27, 2024 20:39
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