Tip Calculator

Community feedback

• 

  Rachel Tschanz•210

  @rtdevcraftPosted 3 months ago

  Hi there! Nice solution!

  Looks good on all sizes and nice separation of concerns! I like the UX of having the reset button with a lighter opacity until it is relevant.

  Consider using semantic HTML like <header> and <main>

  Looks like you have a typo for your aria-label here https://github.com/SecretariatV/FM-TIP-Calculator/blob/main/src/features/layout/tipPanel/index.tsx

  You might search for 'ari' in other areas with CTRL + F (Command + F for OS) just in case you made the same typo elsewhere.

  Not sure if starting with 0 people and an error as default is the best UX.

  Looks like you could change the font to the space mono for the tip buttons and for the bull and number of people inputs.

  0