Skip to content
Submitted over 1 year ago

2rd attempt Product preview card component

Kamu Marvin Baguma•20
@marvo-gift
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 used a display grid. I feel great to say that I am doing the front-end mentor challenge as a practice and learning a lot from it. Any comments about this work is really appreciated especially on where to improve.

Community feedback

  • Omprakash Rahangdale•900
    @OmprakashRPosted over 1 year ago

    Hi, Congratulations!!

    You did a good job.

    I have some Suggestions for this task.

    • remove the margin:400px form you can center a div using flex and grid
    • need to center div with add this approched to main tag flex - body { display: flex; background-color: gainsboro; justify-content: center; align-items: center; min-height: 100vh; }

    using Grid - display: grid; place-content: center; min-height: 100vh;

    change midbox:margin:400px to midbox:margin:0 auto

    I hope this is helpful to you.

    Thank You!!.

    3

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