Skip to content

[Chat] Added Aria Roles to Attachment Components#4731

Merged
jpeng-ms merged 10 commits intomainfrom
john/file-a11y-fix
Jun 17, 2024
Merged

[Chat] Added Aria Roles to Attachment Components#4731
jpeng-ms merged 10 commits intomainfrom
john/file-a11y-fix

Conversation

@jpeng-ms
Copy link
Copy Markdown
Member

What

Addressed the A11y issue where aria roles are missing for attachment card components

Why

bug fix

How Tested

  1. run sample code
  2. attach a file in sendbox
  3. inspect in web tool
  4. check if aria role presents

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2024

Calling bundle size is not changed.

  • Current size: 5020568
  • Base size: 5020568
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2024

CallWithChat bundle size is decreased✅.

  • Current size: 6378098
  • Base size: 6378161
  • Diff size: -63

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2024

Chat bundle size is decreased✅.

  • Current size: 2096903
  • Base size: 2096966
  • Diff size: -63

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2024

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

Lines Statements Functions Branches
Base 26189 / 40997
63.88%
26189 / 40997
63.88%
715 / 1280
55.85%
2110 / 3336
63.24%
Current 26188 / 40997
63.87%
26188 / 40997
63.87%
715 / 1280
55.85%
2079 / 3317
62.67%
Diff -1 / 0
-0.01%
-1 / 0
-0.01%
0 / 0
0%
-31 / -19
-0.57%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2024

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

Lines Statements Functions Branches
Base 50869 / 81957
62.06%
50869 / 81957
62.06%
1035 / 2310
44.8%
3012 / 4925
61.15%
Current 50873 / 81962
62.06%
50873 / 81962
62.06%
1035 / 2310
44.8%
3027 / 4927
61.43%
Diff 4 / 5
0%
4 / 5
0%
0 / 0
0%
15 / 2
0.28%

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

Comment thread packages/react-components/src/components/Attachment/AttachmentCard.tsx Outdated
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@jpeng-ms jpeng-ms enabled auto-merge (squash) June 17, 2024 22:30
@github-actions
Copy link
Copy Markdown
Contributor

@jpeng-ms jpeng-ms merged commit 62155a6 into main Jun 17, 2024
@jpeng-ms jpeng-ms deleted the john/file-a11y-fix branch June 17, 2024 22: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.

6 participants