order-summary-component-main solution

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted about 3 years ago

  • there is a main semantic element - use it instead of the div with the class of main. Semantic matters

  <div class="change">
                         <p> Change</p>
                   </div>
  

  • the above should have been an interactive element - i would suggest an a element
  • the site is responsive , well done. But i think adding some space at the top so that the image does not touch the top

  Marked as helpful

  1

  ---

  

  vishal wankhade•495

  @Vishal-wankhadePosted about 3 years ago

  @ChamuMutezva will do it next time . thanks for the feedback

  0
• 

  dewslyse•3,025

  @dewslysePosted about 3 years ago

  Hello Vishal!! I believe you did well on this challenge. Some things to note:

  • You should consider wrapping your main content in a main tag. You could change <div class="main"> to <main class="main">.

  0