Submitted almost 2 years ago
i used some simple js to finish this site
@mohammadsalih
Design comparison
SolutionDesign
Solution retrospective
i tried to do my best i would like to hear from you all
Community feedback
- @dannyfongdevPosted almost 2 years ago
Your code looks clean and is readable. The only thing I thought in terms of usability, is that you could have disabled the button initially and enabled it after a rating was selected.
You have all the code available to do this. In function active(), you can take away the disabled attribute.
The accessibility issues are easy to take care of. Add a <main> and turn your <h3> into an <h1>.
Good work!
Marked as helpful1@mohammadsalihPosted almost 2 years ago@dannyfongdev thank you so much for your feedback i really apricate it , and i will fix the things you mentioned ❤
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