Responsive order summary component using #HTML #CSS

Community feedback

• 

  yishak abrham•2,150

  @yishak621Posted almost 2 years ago

  Good design but forces the background svg ...use these properties ...specially no-repeat

      background: url(./images/pattern-background-desktop.svg) no-repeat top; 
       background-size: contain; 
       background-position-y: -30%;[to drag background in the y axis]
  

  Marked as helpful

  0
• 

  Sinisa Vukmirovic•1,690

  @SinisaVukmirovicPosted almost 2 years ago

  Hello!

  Looks good to me.

  Just that one error in your report, that's an easy fix. Using <footer> instead of .attribution DIV.

  To further improve as a dev, take a look at "mobile first approach".

  Best of luck!

  0

  ---

  

  Yohannes Teshome•140

  @tiobistaPosted almost 2 years ago

  @SinisaVukmirovic thankyou for your feedback.

  0
• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  To fix your background use background-repeat: no-repeat and also better to use footer tag instead of div where the class is "attribution" 😊

  0