Skip to content
Submitted over 2 years ago

Tip Calculator

sass/scss
Grégory Locigno•50
@gregorylocigno
A solution to the Tip calculator app challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I had a lot of fun with this challenge! The HTML / CSS part was quite easy but the Javascript part took a lot of work, it was not easy.

Thanks to the two people who helped me with the radio button. Without them, I would never have made it!

If you could give me feedback on my code, that would be great!

Code

Please log in to post a comment

Log in

Community feedback

  • P
    Miran Legin•740
    @miranlegin

    Hi Greg,

    first of all congratulations on completing this challenge!

    Using it for a couple of minutes i encountered some problems along the way and i will document it here.

    Starting from top to bottom:

    1. it is possible to enter non-numerical characters into inputs
    2. when i enter first character in the "bill" field there is a validation message in the "number of people" field, not necessary in my opinion
    3. if you enter something in the "bill" and "people" fields there should be a calculation for "total / per person"
    4. if you select "10%" from the predefined values and enter any number in "custom" field 10% is left as active
    5. when all 3 steps are populated ("bill" = 100, "tip" = 10%, "number" = 1) and you want to change the number of people, only "tip amount" is changed and not "total"
    6. if i enter 2 persons, and 200$ next nothing is happening, no calculation is done

    Cheers, Miran

    Marked as helpful

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