Design comparison
Solution retrospective
I'm most proud of the animations I added to the button. I am also proud that the component will respond to small screen sizes and shrink.
What challenges did you encounter, and how did you overcome them?Not many; most of the issues I would have encountered have already been solved with my previous two solutions.
What specific areas of your project would you like help with?Any cool tips and tricks would be appreciated.
Community feedback
- @DavidCasanova83Posted 5 months ago
Hey, very nice work with your TailwindCSS .
Just some advice about accessibility :
List of Accessibility Improvements
-
Add ARIA (Accessible Rich Internet Applications) tags:
- Add ARIA roles for navigation and content elements.
<div class="w-full max-w-80 bg-dark-grey p-6 rounded-lg flex flex-col items-center gap-6" role="contentinfo">
-
Improve image descriptions:
- Add more detailed descriptions for images.
<img alt="Profile picture of Jessica Randall" src="./assets/images/avatar-jessica.jpeg" class="rounded-full h-24 w-24" draggable="false"/>
-
Add navigation attributes to links:
- Add
aria-label
attributes to links to indicate their destination.
<a href="#" class="w-full py-3 bg-grey rounded-lg hover:bg-primary hover:text-off-black transition-colors" aria-label="Jessica Randall's GitHub profile">GitHub</a>
- Add
-
Add a container for the
<main>
tag:- Add a
<main>
tag to indicate the main content region.
<main class="w-full max-w-80 bg-dark-grey p-6 rounded-lg flex flex-col items-center gap-6">
- Add a
-
Use accessible links:
- Ensure that external links open in a new tab while informing users.
<a href="https://www.frontendmentor.io?ref=challenge" target="_blank" rel="noopener noreferrer" class="text-primary hover:text-primary visited:text-primary" aria-label="Frontend Mentor challenge opens in a new tab">Frontend Mentor</a>
Marked as helpful1@BrianHammerPosted 5 months ago@DavidCasanova83 Thanks for the feedback. I especially like the suggestion of opening the link inside a new page. I've looked into your suggested attributes and understand most of them, but I don't understand is the fifth suggestion with
rel="noopener noreferrer"
. What is the benefit of setting these values?0 -
Please log in to post a comment
Log in with GitHubJoin 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