Desktop First Site With React JS and Bootstrap

Community feedback

• 

  LetsFrontend•500

  @B1N4R1Posted over 3 years ago

  Hi Manan,

  The solution looks good, here some improvements:

  • The design approach you took is good but I would suggest using CSS grid for this one, this way you don't need to use that much divs.
  • Also I would suggest using rems to define font-size instead of %
  • The logic for the calculator works great just be carefull with the multiple periods (.....), you should only allow for one period to be inserted.

  Rest of it looks nice, good work!

  Cheers!

  2

  ---

  

  Manan Sahni•290

  @MSS023Posted over 3 years ago

  @B1N4R1 thank you for your suggestion. Will definitely look into that.

  0