HTML and SCSS and Vanilla JS

Community feedback

• 

  Donald•280

  @millidavittiPosted over 2 years ago

  Went through your code an saw this little change you could make. The code below can be written on one line, billValue.value = null; headCount.value = null; tipButtons.value = null; customTip.value = null;

  The one linner code would be: billValue.value=headCount.value= tipButtons.value= customTip.value = null;

  Marked as helpful

  1
• 

  Paola•320

  @paolas771Posted over 2 years ago

  Hello you did a good job. When using this page I’m assuming they are filling it out from top to bottom. To improve your site, instead of having an event listener on the tip buttons you can have it on the number of people input and have your calculate function in there so the user does not have to press the tip button again. So as soon as the user enters number of people the outputs are updated You could also make the number of people box the same size of the bill input box so it’s looks nicer.

  Marked as helpful

  0

  ---

  

  Alucard•870

  @Alucard2169Posted over 2 years ago

  @paolas771 Thanks for your feedback, and yes i am gonna do that now. Thank you

  0