Design comparison
Solution retrospective
Practice is the only way to learn more frontend development.
Community feedback
- @HcmwebsPosted over 2 years ago
Hi Jared,
Great Job! Your solution looks impressive.
With regards to some pointers, of course, if this is ok with you,
- try to landmark the project. Use the HTML5 header, main, footer .... to clear the accessibility issue. Check out this explanation:
https://dequeuniversity.com/rules/axe/4.3/landmark-one-main?application=axeAPI
- I would use the h1- h6 hierarchy for the heading, always in acceding order. Instead of starting with an h2, I would start with an h1, styling it as per the guidelines of the project.
- The heading and the author's name should have a focus state on hover as per the design.
I hope you find these of assistance.
If you require any assistance with the above pointer, please let me do not hesitate to contact me.
Regards,
Hcmwebs
Marked as helpful1 - @shashreesamuelPosted over 2 years ago
Hey good job completing this challenge. Keep up the good work
Your solution looks great however I think that the lower half of the card content needs some margin from the bottom using
margin-bottom
I hope this helps
Cheers Happy coding š
1
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