Skip to content
OverviewSolutions
7
Comments
5
Arun
@ninjabro

All comments

  • Arun•90
    @ninjabroPosted about 1 year ago

    need slight attention for the padding in the text container and parent card width. looks amazing!

    Marked as helpful
    1
  • Arun•90
    @ninjabroPosted almost 2 years ago

    congrats on completing this challenge, did you think of changing grid properties for the mobile screen size?

    @media screen and (max-width: 375px) { .gridbox { grid-template-columns: 1fr; }

    i assume this might let the containers align on top of each other.

    0
  • iris_rose 23•70
    @sawsan-codeSubmitted almost 2 years ago

    hey guys i have a problem with the annual part and i didn't know how to fix it i appreciate any feed-backs about it or the code thank you

    Order-Summary

    1
    Arun•90
    @ninjabroPosted almost 2 years ago

    did you think of putting those annual plan and $59.99 into a div?

    make them as flex container and use flex direction to column.

    0
  • Arun•90
    @ninjabroPosted almost 2 years ago

    hey, may i ask you how did you get the exact size of card? did you use ruler or something? thank you.

    0
  • rymxoxo•60
    @rymxoxoSubmitted almost 2 years ago

    Hello everyone ,

    I'm a little bit disappointed of my desktop result for this challenge, I don't know what happened to my brain to stop thinking for the solution. I did the mobile version first and it went perfectly fine for me but once i started the desktop version, I didn't get the result as the provided preview.

    That's why I need both your feedback and advice to enhance the disaster i have made. CANT WAIT TO READ UR COMMENTS thank you !!!! UPDATE EVERYTHING IS FIXED THANK U ALL

    product-preview-card-component

    2
    Arun•90
    @ninjabroPosted almost 2 years ago

    i might want you to work on the layout first

    size for desktop is too big, reduce the size and you should be good to go!

    i used grid for layout and flex for inside div containers

    you could improve these below listed

    center the card container in body , paddings around the perfume card, button font size, height of button.

    hope this might help you

    0