html, css, vanilla JavaScript

Solution retrospective
Is there any suggestions, please let me know..
Please log in to post a comment
Log in with GitHubCommunity feedback
- @FarisPalayi
Good job completing this challenge👍 Looks good, especially in the mobile👏
Some of my suggestions are:
- 'Learn more' should be an interactive element like
<a>
or<button>
depending on what does it do.(Usually, it's an anchor tag) - Also, try to add some
:focus
state styles because it can improve keyboard ⌨ users experience drastically - Wrap those social media icons in
<a>
tags, since those are links to other webpages. And give it an appropriatearia-label
ortitle
or screen reader only text for better accessibility.
That's all from me. Have fun coding ✨
Marked as helpful - 'Learn more' should be an interactive element like
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