Order summary component

Community feedback

• 

  P

  Kent O'Sullivan•1,870

  @12KentosPosted over 2 years ago

  Hey @arnold-ryan,

  Nice job on the project looks great!

  To fix the issue you have with the background repeating itself like that simply add the following code to the body element.

  background-repeat: no-repeat; background-color: hsl(225, 100%, 94%);

  This will make it so the background no longer repeats itself, as well as set the background color of the page to the same one in the example.

  Hope that helps,

  Keep up the great work!

  0
• 

  Suraj Raju Hadage•670

  @SurajHadagePosted over 2 years ago

  Always use main element and one h1 element. If you are overwhelmed with semantic element just use role attribute like following

  <div role="main"> <div role="heading"></div> <div role="banner"></div> <div role="figure"></div> </div> Don't use section unless you are using h2 to h6 tag insight. Don't upload unnecessary folders and files on github to save server space. Take a look at "https://github.com/SurajHadage/order-summary-component" Tip : always use body { min-width:375px; max-width:1440px; min-height:100vh; } in your every challenge.

  0