Design comparison
SolutionDesign
Solution retrospective
Hello dev,
I keep going to master css, looking forward toy your feedback
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, great work on this one. Desktop layout looks good as well as the mobile layout. The responsiveness is good as well.
Some suggestions would be:
- The price itself, I would not wrap it in its own heading tag. If I will wrap it, it will be included in the
monthly subscription
h3
element, so that users will already know what is the price. It could also bee wrap only in ap
tag, because if the user navigates to themonthly subscription
heading tag, they will know for sure that the next text element would be the price. - The
sign up
should have been usinga
tag and notbutton
. If this were aform
element,button
should be and since it is not, better use linksa
on them. - On the
why us
section. Those list of information below it should have usedul
element. Because like I said, those are list of information on "why you should choose us", right.
Other than those, css looks fine and the whole layout itself. Great work.
0 - The price itself, I would not wrap it in its own heading tag. If I will wrap it, it will be included in the
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