Design comparison
SolutionDesign
Community feedback
- @MuhammadM1998Posted almost 3 years ago
Use form instead of a div. Semantic HTML is always good. You should use labels with inputs, or aria-labels for sake of accessibility. Tip buttons should have a pointer cursor. I think the app should recalculate every time I select a different tip, no just when I change the number of people, even then I should leave the input field only then it calculates.
Try to add event listeners in the script files instead of calling the function in HTML.
Great Work, keep up!
1
Please log in to post a comment
Log in with GitHubJoin 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