Skip to content
Submitted about 1 year ago

Responsive webpage using flex box

Kingsleigh•260
@Kingsleigh-Obi
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I am open to any suggestion or feedback from anyone. Thank You

Community feedback

  • hitmorecode•6,760
    @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 the body. 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 class lower-price and strike 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 GitHub

Join 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