Design comparison
SolutionDesign
Community feedback
- @TH3RIVPosted almost 2 years ago
Hi, @ahmedd-osama!
Your work looks great, but I have a few suggestions to make it even better:
- Your page should contain landmarks, so it makes your code easier to read on what content is what and also helps screen-readers to figure out what is what. In this case your card should be wrapped withing
main
tags and your attribution should be wrapped withinfooter
tags. - Try to avoid using
px
values as much as possible and userem
instead. - Try to start using "custom CSS properties". While this is not essential in this project, it is a good habit to develop.
Hope this helps!
Marked as helpful1@ahmedd-osamaPosted almost 2 years ago@TH3RIV Thanks a lot. I will start to look out for those details next time.
0 - Your page should contain landmarks, so it makes your code easier to read on what content is what and also helps screen-readers to figure out what is what. In this case your card should be wrapped withing
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