Design comparison
Solution retrospective
I created this project a few months ago when I started learning how to code, so the html and CSS are not the best, but I had to wait until my knowledge of javascript was sufficient to complete it. I created a second version with better CSS classes, but I decided to upload this one because I think it's the one that comes closer to the original and, honestly, it reminds me of how far I've come three months later.
The most difficult part was to keep the selected state when clicking on a number or change it if I selected a different one.
Questions: 1.- I know it's a simple component, but would it be okay to use the <article> tag to wrap the text inside the cards? 2.- The javascript works, but is there a way to refactor the code so it looks cleaner?
Any other feedback is more than welcome!
Community feedback
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