-
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
fix: [IOBP-1238] a11y label to the ReceiptListItemTransaction component #6780
Conversation
Jira Pull Request LinkThis Pull Request refers to Jira issues: |
PR Title Validation for conventional commit type✅ All good! PR title follows the conventional commit type. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6780 +/- ##
==========================================
- Coverage 50.94% 50.94% -0.01%
==========================================
Files 1578 1578
Lines 32708 32710 +2
Branches 7374 7426 +52
==========================================
+ Hits 16663 16664 +1
+ Misses 15998 15987 -11
- Partials 47 59 +12
... and 12 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Short description
This PR fixes the accessibility text on the list item transaction component, before this fix, the text was read twice.
List of changes proposed in this pull request
accessibilityLabel
to theListItemTransaction
amountAccessibilityLabel
with the correct value stringHow to test
Inside the payments screen, with the screen reader enabled, check that the transaction list is announced once with the correct displayed data.
Preview
before.webm
after.webm