Design comparison
SolutionDesign
Solution retrospective
any Feedbacks on my project
Community feedback
- @Mr-jawPosted over 1 year ago
Hello there 👋
Great job on completing the challenge
HTML 📄
-
for short texts use
<em>
,<strong>
or<small>
rather than using<p>
which is used for descriptive text. Using Semantic HTML will improve accessibility -
Always provide the
<img>
tag with meaningful and humanly understandable text about what the image is about in thealt
attribute. to improve accessibility.
CSS 🎨
- Use relative units such as
em
orrem
for margin, padding, width, and height. most preferablyrem
for font size. avoid usingpx
since it is an absolute unit.
It is great that you have used CSS custom properties
Overall a Great job you have done
I hope it was useful 😊
KEEP GRINDING
Marked as helpful1 -
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