Order_Summary_Component

Community feedback

• 

  darryncodes•6,370

  @darryncodesPosted about 3 years ago

  Hi Walmanjm,

  You've smashed it, well done!

  • you could clear up your accessibility report though. Change <div class="container"> to <main class="container"> and <div class="attribution"> to <footer class="attribution"> semantic html is reall important
  • also your design should have one <h1> and you shouldn't skip heading levels

  All the best!

  Marked as helpful

  0

  ---

  

  walmanjm•425

  @walmanjmPosted about 3 years ago

  @darryncodes thanks a lot, very much appreciate 👍,

  point to improve :

  • semantic html
  • heading levels

  this is helpfull

  0

  

  walmanjm•425

  @walmanjmPosted about 3 years ago

  @darryncodes

  accessibility report now is clear

  please check updated solution

  0

  

  darryncodes•6,370

  @darryncodesPosted about 3 years ago

  nice one @walmanjm, looks great! Good to see you've implemented the feedback already. You should have the option to refresh the accessibility report to get rid of those errors now you've fixed them.

  0