Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

CSS GRID, FLEXBOX, JAVASCRIPT

@siddhants1

Desktop design screenshot for the Tip calculator app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi, This is my first solution for a challenge. I would like community to give me constructive feedback.

Community feedback

Paola 320

@paolas771

Posted

You did a good job, but the way the user would use this is going down so entering bill first than tip amount than number of people so when they finish and they still do not get total amounts they will be confused and will not know that they have to go back and press tip amount again. The way I did it is I put an event listener on number of people input instead so as soon as user enters number of people input is given. Here is my solution if you want to check it out https://www.frontendmentor.io/challenges/tip-calculator-app-ugJNGbJUX/hub/tip-calculator-app-with-javascript-G00OTLZW4

0

@siddhants1

Posted

@paolas771 Thanks for your feedback, i'll try to improve this up as I've just started learning. I'll try to incorporate the changes that you have suggested. I'm looking forward to more feedback from community.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord