Design comparison
Solution retrospective
- explored responsiveness
- hover effects
Community feedback
- @Kein-InternetPosted 3 months ago
It is responsove and the hover effect is good, nice. Here are some takeaways from this:
-
You must use
rem
instead ofpx
when setting the margin/gap, padding, and font-size so that they accomodate the users defult broswser font-size. Doing this is necessary for making an accessible site. -
On accessibilty, please use appropriate HTML semantics and less
<div>
and<p>
tags, for examlpe use you could wrap your content in a<section>
tag rather than a<div class="left">
tag. One exception is the use of<h2>
tag, you can use<p>
instead as it's just a price, not a title/subsection. -
Use CSS variables to increase readibility and reusibility.
-
You're missing the mobile app image, please add this to your code as to ensure your code accurately matches the design.
Marked as helpful0 -
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