Design comparison
SolutionDesign
Community feedback
- @RioCantrePosted over 2 years ago
Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…
- Instead of
div
, alternatively wrap the whole content with semantic tags, for HTML structures, refer it with this one Semantics - Clean the whitespaces in the code
- Remove unnecessary code to keep it clean
In the brighter side...
- The project is responsive and utilized the details based on the original design
- Well done in using Sass for styling with proper compositions and sizing elements
- The active and hover state of the design is done accordingly
- The JS functionality of the project is done properly, it generates the selected number upon the click event
Above all, The project is awesome! Keep it up!
Marked as helpful1@saulry90Posted over 2 years ago@RioCantre Thanks for your answer!! The comments are very useful to me. I have to check the html
0 - Instead of
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