Skip to content

Multiple A11y fixes to ReactionButton#6033

Open
mgamis-msft wants to merge 11 commits intomainfrom
mgamis/fix-reaction-button
Open

Multiple A11y fixes to ReactionButton#6033
mgamis-msft wants to merge 11 commits intomainfrom
mgamis/fix-reaction-button

Conversation

@mgamis-msft
Copy link
Contributor

What

  • Enhance ReactionButton accessibility by focusing the first emoji button on callout positioning
  • Fix focus moving out of ReactionButton flyout when navigating with Tab key
  • Screen reader announcing position information for the emoji buttons

Why

https://skype.visualstudio.com/SPOOL/_workitems/edit/4322439
https://skype.visualstudio.com/SPOOL/_workitems/edit/4312689
https://skype.visualstudio.com/SPOOL/_workitems/edit/4322423

How Tested

Tested locally
https://github.com/user-attachments/assets/bda9a74e-ba70-4644-a220-dd980ef34f5b

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
Contributor

@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2025

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

Lines Statements Functions Branches
Base 29739 / 47381
62.76%
29739 / 47381
62.76%
810 / 1491
54.32%
2394 / 3842
62.31%
Current 29703 / 47460
62.58%
29703 / 47460
62.58%
814 / 1496
54.41%
2414 / 3858
62.57%
Diff -36 / 79
-0.18%
-36 / 79
-0.18%
4 / 5
0.09%
20 / 16
0.26%

@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2025

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

Lines Statements Functions Branches
Base 58580 / 95109
61.59%
58580 / 95109
61.59%
1180 / 2709
43.55%
3417 / 5799
58.92%
Current 58605 / 95229
61.54%
58605 / 95229
61.54%
1184 / 2714
43.62%
3443 / 5826
59.09%
Diff 25 / 120
-0.05%
25 / 120
-0.05%
4 / 5
0.07%
26 / 27
0.17%

@github-actions
Copy link
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

@github-actions
Copy link
Contributor

📦 CallWithChat bundle size

Bundle Base Size (bytes) Current Size (bytes) Change Delta (bytes)
build.bundle.js 7758336 7759115 ⚠️ increased 779
450.bundle.js 4470 4470 ➖ unchanged 0
580.bundle.js 12407938 12407938 ➖ unchanged 0
727.bundle.js 372564 372564 ➖ unchanged 0
82.bundle.js 28177 28155 ⬇️ decreased -22
91.bundle.js 4118 4118 ➖ unchanged 0

Total change: +757 B ⚠️

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

📦 Calling bundle size

Bundle Base Size (bytes) Current Size (bytes) Change Delta (bytes)
build.bundle.js 6767681 6769228 ⚠️ increased 1547
580.bundle.js 12407925 12407925 ➖ unchanged 0

Total change: +1.5 KB ⚠️

@github-actions
Copy link
Contributor

📦 Chat bundle size

Bundle Base Size (bytes) Current Size (bytes) Change Delta (bytes)
build.bundle.js 1793916 1794092 ⚠️ increased 176
398.bundle.js 375829 375829 ➖ unchanged 0
450.bundle.js 4454 4454 ➖ unchanged 0
82.bundle.js 28161 28139 ⬇️ decreased -22
91.bundle.js 4102 4102 ➖ unchanged 0

Total change: +154 B ⚠️

@github-actions
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments