Skip to content

[A11y][bugfix] Add Role to SvgWithWordWrapping #5127

Merged
edwardlee-msft merged 4 commits intomainfrom
elee/bugfix-a11y-svg-text-announcing-graphic
Sep 6, 2024
Merged

[A11y][bugfix] Add Role to SvgWithWordWrapping #5127
edwardlee-msft merged 4 commits intomainfrom
elee/bugfix-a11y-svg-text-announcing-graphic

Conversation

@edwardlee-msft
Copy link
Copy Markdown
Contributor

@edwardlee-msft edwardlee-msft commented Sep 5, 2024

What

  • Add Role to SvgWithWordWrapping
  • Add heading role in ConfigurationPage use
    Video of the Fix in action. Turn up audio to hear. Go to the Why section to see the original problem in video.
Working.corectly.mov

Why

  • A11y bug where when using Windows narrator the SvgWithWordWrapping announces "Graphic" when focused. Should not announce Graphic and should announce heading.
    Graphic announced. Turn up audio to hear it in the video:
Graphic.announced.mov

How Tested

Window 11 Devbox

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 Sep 6, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

Chat bundle size is not changed.

  • Current size: 2112203
  • Base size: 2112203
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

Calling bundle size is not changed.

  • Current size: 8139365
  • Base size: 8139365
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

CallWithChat bundle size is not changed.

  • Current size: 8139379
  • Base size: 8139379
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

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

Lines Statements Functions Branches
Base 26429 / 42318
62.45%
26429 / 42318
62.45%
723 / 1312
55.1%
2089 / 3346
62.43%
Current 26422 / 42318
62.43%
26422 / 42318
62.43%
723 / 1312
55.1%
2097 / 3354
62.52%
Diff -7 / 0
-0.02%
-7 / 0
-0.02%
0 / 0
0%
8 / 8
0.09%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

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

Lines Statements Functions Branches
Base 53951 / 88340
61.07%
53951 / 88340
61.07%
1076 / 2465
43.65%
3167 / 5234
60.5%
Current 54026 / 88342
61.15%
54026 / 88342
61.15%
1076 / 2465
43.65%
3172 / 5239
60.54%
Diff 75 / 2
0.08%
75 / 2
0.08%
0 / 0
0%
5 / 5
0.04%

@edwardlee-msft edwardlee-msft enabled auto-merge (squash) September 6, 2024 16:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 6, 2024

@edwardlee-msft edwardlee-msft merged commit 0cea67d into main Sep 6, 2024
@edwardlee-msft edwardlee-msft deleted the elee/bugfix-a11y-svg-text-announcing-graphic branch September 6, 2024 17:56
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.

4 participants