Order summary component

Community feedback

• 

  Adriano•38,390

  @AdrianoEscarabotePosted almost 2 years ago

  Hi Vlad, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

  To prevent the background image from breaking at higher resolutions, we can prevent this in two different ways:

  1. Add a background-repeat: repeat-x;, the image will repeat on the horizontal axis, preventing it from breaking.
  2. Add a background-size: 100% 50vmin;, the 50vmin will set its height as the page target, and 100% will make it stretch on the horizontal axis.

  Feel free to choose one of the two!

  To improve the code structure wrap this div:

  <div class="attribution">

  with the semantic tag footer

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1

  ---

  

  Vlad•60

  @VlaaadddPosted almost 2 years ago

  @AdrianoEscarabote thank you very much.

  1

  

  Adriano•38,390

  @AdrianoEscarabotePosted almost 2 years ago

  @Vlaaaddd no problem, keep coding!!

  0
• 

  Mortada•250

  @xTern44Posted almost 2 years ago

  Everything looks fine except for the positionning of the container. I suggest you try these adjusments:

  body { height: 100vh; position: relative; }

  .container { position: absolute; top: 50%; left: 50%; transform: translate(-50%,-50%); }

  Marked as helpful

  1