order-summary-component-main

Community feedback

• 

  Adriano•38,390

  @AdrianoEscarabotePosted almost 2 years ago

  Hello Oluwatobi, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:

  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!

  This page doesn't even need a "h1" because it is just a component, as you are aware. because we don't know how crucial the site's other elements will be! However, it's always advisable to avoid accessibility errors, so I believe it would be beneficial for you to include a "h1" in this component. This is also a good practice for when you are creating larger websites, so don't worry if you forget to include a "h1".

  <h1>Order Summary</h1>

  The remainder is excellent.

  I hope it's useful. 👍

  0