NFT preview card component (flex CSS)

Solution retrospective
Any suggestions on how I can improve and reduce unnecessary code are welcome!
Thank you. πβοΈ
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Mr-jaw
Hello there π
Great job on completing the challenge
HTML π
- replace
<h2>
with<h1>
sicne it is the main heading of the component
CSS π¨
-
Use relative units like
em
orrem
for margin, padding, width, and height. and most preferablyrem
for font sizes. avoid usingpx
which is an absolute unit -
Always provide meaningful and humanly understandable text about what the image is about in the
alt
attribute in the<img>
tag.
NOTE: It's considered best practice to write all your CSS in a separate file and link it to the HTML
I hope this was helpful π
KEEP GRINDING
Marked as helpful - replace
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