Design comparison
SolutionDesign
Solution retrospective
I am open to any suggestion or feedback from anyone. Thank You
Community feedback
- @hitmorecodePosted about 1 year ago
Nice well done. I have a few suggestions
- Make it a habit of using css reset, if you are not familiar with css reset, just google it. Here is a simple css reset you can always use
* { margin: 0; padding: 0; box-sizing: border-box; }
- Make it a habit of using
min-height: 100vh;
on thebody
. This is helpful in making responsive pages. - If you add flexbox on the
body
, you can remove the margin on the.container
. - If you place the
<p>
tags with classlower-price
andstrike
inside a<div>
and apply flexbox on this<div>
you'll be able to align both prices. - You have an empty
<p>
you can remove it, it's not doing a thing. - For next challenges try to build your html structure like this
<body> <main> /* place everything else in here */ </main> </body>
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