Order Summary Challenge HTML/CSS

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Hey there,

  Your solution looks great, well done ^^ There only a few small things you could perhaps improve:

  1. The max-width on the card could be 2-3 rems more.

  2. I don't think it's necessary to wrap the payment and the cancel order buttons in a form. It's not really a form, a div will do the job. Sometimes a simpler solution works better ^^

  3. For the "Annual Plan" text you could choose a different heading level than what you used for the "Order Summary". Right now, they're both h2 but they should be logically different.

  4. There's an unnecessary 100% width on the main element. Nothing changes if you remove it. In the same place, you should swap the height with min-height: 100vh, for responsiveness.

  Apart from all these, you did good, the styling looks fine - Annual Plan section included! Happy coding ^^

  Marked as helpful

  0

  ---

  

  Michael Waaler•230

  @10highPosted over 2 years ago

  Thanks for the detailed feedback @FluffyKas - that's really useful.

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hello 10high ! Congo 👏 on completing this challenge

  Let's look at some of your issues, shall we:

  • As per the preview i can see that button shadow is more in your solution that there original design so maybe reduce it a bit

  • Wrong font-weight for "Change"

  happy Coding😀

  0