Design comparison
SolutionDesign
Community feedback
- @crsimpson5Posted over 1 year ago
Hey Ossy, great job on your solution. The styling looks great and you've matched the design really well.
A couple things you can improve:
- The inputs start with a value of "0", so if I select the input and type "2" it becomes "20". You can initialize the state with
null
to prevent this. - The inputs would be better suited for
type="number"
. That way the app doesn't break if a letter is entered. If you want to remove the arrows for a number input here's a link on how to do that: https://www.w3schools.com/howto/howto_css_hide_arrow_number.asp. - There's some overflow around 630px screen size. One solution would be to allow the flex container to wrap.
Keep up the good work! 😃
Marked as helpful0@ossycodePosted over 1 year ago@crsimpson5 Thanks for the helpful feedback. Implemented the changes now
0 - The inputs start with a value of "0", so if I select the input and type "2" it becomes "20". You can initialize the state with
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