Flexbox, Add Event Listener, ParseFloat,

Community feedback

• 

  Ayush Nath•360

  @Beats-AyushPosted over 2 years ago

  Hello nice attempt at this challenge. Some points where there can be improvement -

  • Instead of setting <input type='number' />, maybe use <input type='text' /> or simply remove the spinner arrows to improve the UI.
  • Upon Reset, the custom input is not being cleaned up.
  • Look out for the NaN values which are coming up whenever the custom input gets cleared up and then focused again.

  0

  ---

  

  Jason•90

  @Mister-ZengPosted over 2 years ago

  @Beats-Ayush using type number allows me to not write function to ensure it is a number. I also removed the spinner from the UI but can still be used as an option. The custom input, I set it as a default of 1 as it is the minimum.

  0