Responsive solution using SCSS Grid and Flexbox

Solution retrospective
Hey guys,
hope for feedback :)
Please log in to post a comment
Log in with GitHubCommunity feedback
- @fernandolapaz
Hi again 🙂, perhaps some of this may interest you:
HTML:
- When there is a hover state over an element it means that it is interactive, so there must be an interactive element around it (like a link or a button). So, we should use a
<a>
or<button>
to wrap the image (depending on what happened when clicking on it). Also 'Equilibrium #3429' should be wrapped with the<a>
tag (inside the <h1>).
CSS:
- The trick of setting the
font size: 62.5%;
is no longer commonly recommended because it can cause issues with accessibility and readability. Also, it’s good to get used to thinking directly in relative units like rem or em since they are the most suitable for most cases.
I hope it’s useful : )
Regards,
Marked as helpful - When there is a hover state over an element it means that it is interactive, so there must be an interactive element around it (like a link or a button). So, we should use a
- @rohitd99
Hi Rhinozer0s
Congrats on completing the challenge.
I have a suggestion which might interest you :
- You have used
div
for the card, instead ofdiv
you can usemain
which is a semantic element. Read more on semantic elements here.
Hope it helps you. Happy coding.
Marked as helpful - You have used
- @fernandolapaz
Hi 👋,
It seems that the link to the code on GitHub leads to the files of another challenge... 🙂
Regards,
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