-
Notifications
You must be signed in to change notification settings - Fork 105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: [IOBP-1190] Add a11y announcement inside the payment method details screen #6776
chore: [IOBP-1190] Add a11y announcement inside the payment method details screen #6776
Conversation
PR Title Validation for conventional commit type✅ All good! PR title follows the conventional commit type. |
Jira Pull Request LinkThis Pull Request refers to Jira issues: |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6776 +/- ##
==========================================
- Coverage 50.96% 50.95% -0.01%
==========================================
Files 1578 1578
Lines 32697 32699 +2
Branches 7366 7416 +50
==========================================
Hits 16663 16663
+ Misses 15987 15978 -9
- Partials 47 58 +11
... and 13 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Short description
This PR adds a more detailed
a11y
announcement when opening the payment method details screen.List of changes proposed in this pull request
useOnFirstRender
hook to announce the detailed description of the screen;ignoreAccessibilityCheck
in theuseHeaderSecondLevel
hook;How to test
With the SR enabled, open the payment method details screen from the "Wallet" section and check that the detailed description is correctly announced