Skip to content

[A11y bugfix] reaction item aria narration#4635

Merged
dmceachernmsft merged 4 commits intomainfrom
dmceachernmsft/reaction-a11y-naming
May 21, 2024
Merged

[A11y bugfix] reaction item aria narration#4635
dmceachernmsft merged 4 commits intomainfrom
dmceachernmsft/reaction-a11y-naming

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

Add aria labels to each of the reaction items

Why

Allows for the different narrators to read out each name

https://skype.visualstudio.com/SPOOL/_workitems/edit/3674307

How Tested

Validated locally

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 17, 2024

Calling bundle size is increased❗.

  • Current size: 4845659
  • Base size: 4845640
  • Diff size: 19

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 17, 2024

Chat bundle size is not changed.

  • Current size: 2056542
  • Base size: 2056542
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 17, 2024

CallWithChat bundle size is increased❗.

  • Current size: 6170293
  • Base size: 6170274
  • Diff size: 19

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 17, 2024

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

Lines Statements Functions Branches
Base 25829 / 40006
64.56%
25829 / 40006
64.56%
710 / 1255
56.57%
2079 / 3266
63.65%
Current 25832 / 40006
64.57%
25832 / 40006
64.57%
710 / 1255
56.57%
2081 / 3267
63.69%
Diff 3 / 0
0.01%
3 / 0
0.01%
0 / 0
0%
2 / 1
0.04%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 17, 2024

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

Lines Statements Functions Branches
Base 50264 / 80229
62.65%
50264 / 80229
62.65%
1028 / 2278
45.12%
2945 / 4825
61.03%
Current 50264 / 80230
62.64%
50264 / 80230
62.64%
1028 / 2278
45.12%
2964 / 4836
61.29%
Diff 0 / 1
-0.01%
0 / 1
-0.01%
0 / 0
0%
19 / 11
0.26%

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) May 21, 2024 17:28
@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit c98610a into main May 21, 2024
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/reaction-a11y-naming branch May 21, 2024 17:42
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