Checkout page of a music application

Community feedback

• 

  Dmitry•375

  @dmitrymitenkoffPosted about 3 years ago

  Well done, Ranjan. The component looks really great.

  I think your code base is well structured, and it was easy for me to follow the logic. To make it a little bit more succinct, you could try using CSS custom properties.

  One thing I'd suggest is not skipping heading levels in your HTML. For example, I'd use <h1> for the Order Summary and <h2> for the Annual plan. More information is here and here.

  Also, in mobile view, there's a little bit of white space at the bottom of the viewport visible. I'd suggest uncommenting the height property on the body element in your CSS - this should rid of the white gap at the bottom.

  Other than these two minor things, great job and happy coding :)

  Cheers

  Marked as helpful

  0

  ---

  

  Ranjan Verma•10

  @Ranjan-Kumar-VermaPosted about 3 years ago

  @dmitrymitenkoff Thank you for taking your valuable time and giving this feedback, this is very helpful for me and I will correct this changes suggested by you. Have a nice day :)

  0