Skip to content
OverviewSolutions
3
Comments
2
Abdelrahman fares
@bedofares

All comments

  • Abdelrahman fares•90
    @bedofaresPosted about 1 year ago

    Good job ! I see that you have the padding-right & left as 0 which I guess if you set it to 10px same as for the top and bottom padding it will look nicer, you can also play around with the padding until you see it looks good.

    I would suggest removing the padding from the .part-left on the mobile screen and increasing the width to 95% it will also look way better aligned.

    like this :

    @media screen and (max-width: 639px)
.part-left {
    width: 95%;
    padding: 0 0;
}
    Marked as helpful
    0
  • Abdelrahman fares•90
    @bedofaresPosted about 1 year ago

    It looks very nice and I like how responsive it is :) . I have only one comment regarding centering it vertically & horizontally. You can achieve this using flexbox .

    what you need to change is :

    body{
   min-height: 100vh;
   display: flex;
   align-items: center;
   justify-content: center;
}

    and remove the margin-top from the order-summary div.

    I hope this helps :)

    Marked as helpful
    1