Skip to content
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(a11y): enhance navigation and fix ds nav mobile focus #1151

Merged
merged 7 commits into from
Dec 20, 2023

Conversation

Fupete
Copy link
Collaborator

@Fupete Fupete commented Dec 19, 2023

Fix: #1147

We will need to investigate further how to improve the navscroll component of Bootstrap Italia that we are currently repurposing for the sidebar of the design system. We will open an issue as soon as this is merged.

This PR fix the urgent "focusable" part, now it is. However the keyboard "pattern" is still not perfect.

Copy link
Contributor

github-actions bot commented Dec 19, 2023

⚡ Lighthouse report for the home page 🏠

Category Score
🟠 Performance 61
🟢 Accessibility 100
🟢 Best practices 100
🟢 SEO 100

See the full report...

Other pages

@Fupete Fupete requested review from bfabio and astagi December 19, 2023 11:54
@Fupete Fupete marked this pull request as ready for review December 19, 2023 11:55
@Fupete Fupete changed the title fix(a11y): enhance navigation fix(a11y): enhance navigation and fix ds nav mobile focus Dec 19, 2023
@Fupete Fupete merged commit 1b83f3e into main Dec 20, 2023
4 checks passed
@Fupete Fupete deleted the fix-enhance-a11y-navigation-202312 branch December 20, 2023 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Sidebar navigation does not take focus on mobile
2 participants