Skip to content

chore(IT-Wallet): [SIW-3829] Add accessibility announcements for accordion state changes in claims#8150

Open
DavideOnano wants to merge 1 commit into
masterfrom
SIW-3829-fix-accessibility-claims-accordion
Open

chore(IT-Wallet): [SIW-3829] Add accessibility announcements for accordion state changes in claims#8150
DavideOnano wants to merge 1 commit into
masterfrom
SIW-3829-fix-accessibility-claims-accordion

Conversation

@DavideOnano
Copy link
Copy Markdown
Collaborator

Short description

This PR add announce for accessibility when the claims accordion state change on tap

List of changes proposed in this pull request

  • Add announceForAccessibility() function while the user tap on accordion to open or close it, so the screen reader announce the value during the action

How to test

  1. Start a remote presentation
  2. Tap on the claims accordion and verify that TalkBack announce the correct value for opened or closed accordion

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

Jira Pull Request Link

This Pull Request refers to Jira issues:

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.46%. Comparing base (7a7d728) to head (fbc6658).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
...tion/common/components/ItwClaimsSelector/index.tsx 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #8150       +/-   ##
===========================================
+ Coverage   62.38%   88.46%   +26.07%     
===========================================
  Files        1945        1     -1944     
  Lines       36779       26    -36753     
  Branches     6091        5     -6086     
===========================================
- Hits        22946       23    -22923     
+ Misses      12477        0    -12477     
+ Partials     1356        3     -1353     
Files with missing lines Coverage Δ
...tion/common/components/ItwClaimsSelector/index.tsx 88.46% <50.00%> (-3.21%) ⬇️

... and 1944 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c50aab...fbc6658. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DavideOnano DavideOnano marked this pull request as ready for review June 4, 2026 10:28
@DavideOnano DavideOnano self-assigned this Jun 4, 2026
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.

1 participant