Responsive Order Summary using Sass

Community feedback

• 

  Adriano•38,160

  @AdrianoEscarabotePosted about 2 years ago

  Hi Odiesta, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  To improve the page design, we can do this:

  .container {
      /* height: 90vh; */
  }
  

  it is not necessary to use a fixed height in the component!

  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!

  The rest is great!**

  I hope it helps... 👍

  Marked as helpful

  1