tip-calculator sass and debounce function

Community feedback

• 

  Isabela Oliveira•310

  @oliveridsPosted about 3 years ago

  Hey, your code is really neat and works just great!

  I'd suggest using the "input" event instead of "keyup" if you want a more imediate trigger, but that's not necessary if you don't want it.

  Also, for better usability, I suggest not letting the calculator do its thing until the person types the number of people. Your code says that when the value of the input is zero, to use "1" as default value. However, a random user can't see that, and they might think there's an error. I think something like this would be enough for you:

  if (input.value.length) { ..function.. }

  Nice job!!

  Marked as helpful

  0

  ---

  

  VladimirBrscic•775

  @VladimirBrscicPosted about 3 years ago

  @oliverids Thanks a lot for your suggestions!

  0