Design comparison
SolutionDesign
Solution retrospective
For the button, I used an image within a button as it seemed easier to style. Is it correct to have the icon as an image within the button or would setting it as the background of the button be better? Any other feedback also welcomed.
Community feedback
- @rvarunr5Posted 9 months ago
Code looks great. You're missing one edge case, if the user enters
12-12-2024
the result is an empty year which shouldn't be the case. It should give out an error saying the date shouldn't be in the future.Marked as helpful0@cwatsoPosted 9 months ago@rvarunr5 Good catch! I have updated the code to account for this and any other future dates within current year.
0
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