-
Notifications
You must be signed in to change notification settings - Fork 106
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-1179] a11y
banner focus order
#6725
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 ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6725 +/- ##
=======================================
Coverage 50.11% 50.11%
=======================================
Files 1557 1557
Lines 32468 32466 -2
Branches 7356 7360 +4
=======================================
- Hits 16272 16271 -1
+ Misses 16144 16143 -1
Partials 52 52
Continue to review full report in Codecov by Sentry.
|
Short description
This pull request updates the @pagopa/io-app-design-system to solve the accessibility focus order of the
Banner
componentList of changes proposed in this pull request
@pagopa/io-app-design-system
package from version4.5.5
to4.5.6
SwitchLabel
AnimatedMessageCheckboxShowroom
componentHow to test
Pagamenti
sectionTab
andShift + Tab
keys, the switch logic for theBanner
component correctly moves focus (Text -> CTA -> Close Icon)Preview
Screen.Recording.2025-02-17.at.17.00.22.mov