
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@loki-pepe
I can't really judge your HTML and CSS as the app was written using tools I'm not acquainted with, but the good thing is the page is responsive and handles varied font-size settings well.
As for the app itself, the visuals are a bit off, and more importantly, it is a bit buggy:
- the calculator should be empty by default
- there should be error messages for different errors:
- the inputs probably shouldn't accept negative numbers
- the bill input probably shouldn't accept a number with more than two decimals (there are only 100 cents in a dollar)
- for some reason the inputs delete their content when I try to put a non-number into them (apart from the custom tip input). This is a bad user experience. The input should probably allow an invalid character, but then show an error message (or at the very least, not delete the valid input).
Generally, try and keep true to the original design and always try to think about ways a user might use (and break) your app. A few CSS tweaks and a few more lines of JavaScript would greatly help your app. Keep at it!
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