simple ordering page

Community feedback

• 

  Clément Creusat•1,655

  @ccreusatPosted about 3 years ago

  Hey ! Nice work.

  You could replace :

  • div.main-container with a <main> tag
  • div.description with a <p> tag
  • div.annual-plan-text with a <strong> tag
  • div.money with a <span> tag

  You could add :

  • type="submit" on your button tag as it would submit the form

  By doing so, you could replace div.text-container by a <form> tag

  Marked as helpful

  0

  ---

  

  Lealem Birhanu•75

  @Aklog-1Posted about 3 years ago

  @ccreusat thanks, just changed what you told me.

  1
• 

  Heath Allred•35

  @Heathallred4Posted about 3 years ago

  The only thing I would do differently, is use the <main> tag in place of the <div class=main-container>. And possibly use <footer> for the payment and cancel buttons. Those are the only two I noticed, but I'm fairly new at all of this as is. It looks really clean though.

  Marked as helpful

  0

  ---

  

  Lealem Birhanu•75

  @Aklog-1Posted about 3 years ago

  @Heathallred4 all right, thank you!

  0