order-summary-component-main

Community feedback

• 

  ROCKY BARUA•1,090

  @DrougnovPosted over 2 years ago

  Hello @pharaohmak, good job on the design. Still, you can make a lot of improvements to your codes. Here are some suggestions for you:

  • Your HTML isn't semantic. To fix that you can use some tags like header main section footer instead of div.
  • The card should be in the center of the page. Add justify-items: center on the container to move it to the center.
  • The background image is not aligned well. To fix it use

     hsl(225, 100%, 94%);
    background-size: contain;
    background-position-y: -15%; ```
    

  on the body.

  • You have only downloaded font-weight 200. That's why the heading isn't bold. Re-download the font or use this @import url("https://fonts.googleapis.com/css2?family=Red+Hat+Display:wght@500;700;900&display=swap"); in your CSS.

  Have a good day my friend :)

  Marked as helpful

  0

  ---

  

  Makram M Ibrahim•130

  @pharaohmakPosted over 2 years ago

  @Drougnov thank you for the feedback, very helpful!

  0