Design comparison
Solution retrospective
Hey guys,
hope for feedback :)
Community feedback
- @fernandolapazPosted over 1 year ago
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 helpful1@Rhinozer0sPosted over 1 year agothanks @fernandolapaz,
I just added the links
can you tell me more about the issue with the trick of setting
font-size: 62.5%;
and name a source were i can read about? I never had any problems until now.0@fernandolapazPosted over 1 year ago@Rhinozer0s
Of course, here I leave you a couple of posts that refer to that topic:
Josh Comeau - The 62.5% trick π
FEDMENTOR - Should I change the default HTML font-size to 62.5%? π
I hope you find it useful,
Regards
1 - 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
- @rohitd99Posted over 1 year ago
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 helpful0@Rhinozer0sPosted over 1 year agoThanks @rohitd99 for your advice!
I thought about that, but decided to went with the div because if there were several such cards, a div would make more sense.
But especially in this case i agree with you.
0 - You have used
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