Skip to content
Submitted over 3 years ago

Responsive Tip Calculator

Tomiwa Orimoloye•1,000
@tomiwaorimoloye
A solution to the Tip calculator app challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Any feedback is welcomed. Especially on how to refactor my javascript code to make it more maintainable & reusable.

Code

Please log in to post a comment

Log in

Community feedback

  • Dmitry•375
    @dmitrymitenkoff

    Hi Tomiwa,

    This is an awesome project - great job! I'm not a JS expert, but your code looks maintainable, you avoid repetition, provide good (and useful) comments and separate the concerns. I've just recently completed the same challenge, and I was really interested in the way you implemented the Tip selection section. I used (and struggled quite a bit with) radio inputs. I notice you opted into using the real button element with data value attributes. This is such a cool way of doing the 'select' section. I wonder if there are any accessibility issues with this approach?

    Once again, well done!! Cheers

    Marked as helpful
  • SivaKotari•135
    @sivakotari

    @Tomiwa Orimoloye, UI and code look so clean and satisfying. Great work!! I've also worked on this project.

    One observation - input fields are accepting any text, why don't you use input type number! And what does caret.js do here?

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