Order Summary

Community feedback

• 

  Bernardus•1,095

  @BernardusPHPosted over 2 years ago

  -hello BINGERMINN, If you are talking about using a better method for the body's background, I did the following on your site, so see for yourself:

  body{
  /* background-position: center top; */
  background-size:contain;
  }
  

  -See if it helps.

  -Also just some advice but try adding an:

  @media(max-width:430px){
     .main-card{
        width:90%;
      }
  }
  

  • for mobile( just fix some of the overflowing that will happen)

  Marked as helpful

  2

  ---

  

  Bingerminn•240

  @yallsobadPosted over 2 years ago

  @BernardusPH Thanks for the feedback!

  I was able to simplify it slightly using:

  background-image: url(images/pattern-background-desktop.svg); background-size: contain; background-repeat: no-repeat; background-color: var(--Pale-blue);

  1
• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello there! 👋

  Regarding the background image for the body element, if you see on the design comparison, the background image has perfectly matched the design. So, I don't think there's an issue. 🙂

  Now, about the cart-item section, you've done a great job with it. I used flexbox and margin when I was building it.

  Some feedback from me.

  • Alternative text for images should not contain any words that related to image (e.g. picture, photo, logo, icon, graphic, avatar, etc). It's already an image element so the screen reader will pronounce it as an image.
  • The illustration and the icon are decorative images so leave the alt empty.
  • I suggest removing the div that wraps anchor tags and button and styling the element itself directly. This will make the HTML looks cleaner and more simple.
  • Always specify the type of the button. In this case, set the type of them as type="button". It's going to prevent the button from behaving unexpectedly.

  That's it! Hope this helps. 😊

  Marked as helpful

  1

  ---

  

  Bingerminn•240

  @yallsobadPosted over 2 years ago

  @vanzasetia It does, thank you!

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @yallsobad You're welcome! 😄

  0