Order summary challenge

Community feedback

• 

  Omar M.•270

  @OmarMAttia7Posted almost 3 years ago

  Congratulations on finishing the challenge, Good job :).

  I'd like to point out a couple of problems with your HTML markup:

  1. Your component should be wrapped inside a <main> element. Every HTML document should have a <main> element.
  2. The title of your document - in this case Order Summary - should be put inside an <h1> element, also every HTML document should have one.

  Marked as helpful

  0
• 

  Attila•290

  @unexunderscorePosted almost 3 years ago

  Hi there! Great work, just keep going!

  Some remarks:

  1. Title (Order Summary) and the Annual Plan text should be different color (bluish).
  2. If you hover over your mouse on the change link, you should not see underline. (text-decoration: inherit; )
  3. Where the music icon and annual plan text etc. There should be a very light gray box around, check the design picture, you can see slightly.
  4. Box shadow under the card layout and the button. (Sample: box-shadow: 0 8px 16px 0 rgba(0,0,0,0.2);)

  Keep coding!

  Marked as helpful

  0