Tip Calculator using CSS Grid, Flexbox, Vanilla JS

Community feedback

• 

  P

  Ken•4,915

  @kens-visuals

  Posted about 3 years ago

  Hey @kenreibman 👋🏻

  I looked at your code and I can see what you're missing. You're missing validation, if you take a look at my solution you'll see that you can't input letters, a bunch of zeros, symbols, and invalid value in general. In your case, you're only checking if the input is filled then calculate the number, but you need to consider the edge cases. Other than that, everything looks and works perfectly. Nice job, and sorry for the late response.

  12

  Marked as helpful
• 

  IRVINE MESA•1,855

  @DrMESAZIM

  Posted about 3 years ago

  Well done @kenreibman ,the HTML and CSS code you used is very good .I would like for you to take note of this in the near future

  1.The code repository URL was incorrectly submitted in your solution .For assistance to correctly do that , please consider watching my YouTube video on this link https://www.youtube.com/watch?v=BW0FCFV323s 2. Using coding skills it would be nice to limit the decimal places on the the "Bill" amount. 3.The "Number of people" integer should not have decimal units .

  01