Order component

Community feedback

• 

  Deepak Parmar•490

  @deepak-parmarPosted over 2 years ago

  @kaminskimaciej, visually your works looks great!

  First of all, you need to renew the screenshot and regenerate the report of your submission.

  After then, you just have to eliminate some of the accessibility issues...

  • Heading elements should always be used in order from h1-h6. For your .title element use h1 tag instead of h2, it won't change anything in the design.
  • The same error is also caused by the h5 tag later in your code. I suggest not using h5 and use <p> tag for it, since it is more like a label than a header.
  • Next, you need to wrap your whole .container element inside the landmark tag <main>, cause all pages must have a landmark element.

  You code looks decently clean, I suggest using some code formatter to automate that part.

  I hope this clears up the issues. 👍

  0