Tip Calculation app

Community feedback

• 

  James.•390

  @JimztechPosted 4 months ago

  great work Casper. but the reset button only clears the input values and not the output value. to clear all values, you'd have something like this in your js for ur reset btn.

      // Reset outputs
      resetButton.addEventListener("click", () => {
        amountOutput.textContent = "$0.00";
        totalOutput.textContent = "$0.00";
     });
  

  This being collected as a dom value. you can check my js code in my own project. But all still great work.

  My bro just checked your code now, please use a form element, in your html, whenever you are dealing with

  <input type="text">
  

  Thanks for reading.

  Marked as helpful

  1

  ---

  

  Erkant•730

  @erntTt94Posted 4 months ago

  @Jimztech What is the point of using form without a submit button? He is not going to send the date anyways.. i don't think its a must to use a form element when there is input element...

  You need to validate your code... there is really too much bugs: --for example: i choose bill: 20, people 2, 15% and nothing happens.. i need to click once more on the %.. than when i delete bill and give it 2 it stays 0, but if i delete it than it shows $2.10.. or if i change people to 2 and bill to 1 it shows infinity.. i really dont know what is going on..

  Marked as helpful

  1