Responsive Huddle Landing Page

Solution retrospective
Please share your feedback. It would be appreciated.
- Is there something to improve?
Please log in to post a comment
Log in with GitHubCommunity feedback
- @FluffyKas
Hey, it's a really great solution! I only have a few small suggestions to make:
-
At the ready section in mobile view, you could add some inline padding to the title so it doesn't touch the edge of its container.
-
"illustration" in unfortunately not a functional alt. If you consider them decorative only, just leave them empty! This article is very helpful if you're not sure about how to write good alt texts: (https://axesslab.com/alt-texts/)
-
Your social icons have aria-hidden on them. They are not images though, they are links and probably you don't want users using screen readers not to have access to features in your website. It's better to use aria-labels here, so someone with a screen reader will know where the link leads!
Marked as helpful -
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord