#CX-13613 : Web - Change State: Screen Readers: Unnecessary and verbose announcement for the opened dialog

momentum-design / momentum-ui

Momentum UI is a collection of UI libraries for implementing Momentum Design into web applications and websites.

https://momentum.design

MIT License

199 stars 204 forks source link

#CX-13613 : Web - Change State: Screen Readers: Unnecessary and verbose announcement for the opened dialog #1650

Closed nithishkumar98 closed 4 months ago

nithishkumar98 commented 4 months ago

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:

Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.12%. Comparing base (ed7b59c) to head (808a2a9). Report is 5 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #1650 +/- ## ========================================== - Coverage 94.17% 94.12% -0.05% ========================================== Files 134 134 Lines 19302 19301 -1 Branches 2986 2985 -1 ========================================== - Hits 18178 18168 -10 - Misses 1124 1133 +9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.