Submitted over 2 years ago
Pure Css, Html, JavaScript, Linters are used to remove errors
@developerwaleed
Design comparison
SolutionDesign
Solution retrospective
Hey Guys 👋, I completed this challenge and I want you guys to please check and recommend me what I would have improved in the project. Thankyou
Community feedback
- @Bayoumi-devPosted over 2 years ago
Hey Waleed, Congratulations on completing this challenge...
My suggestions:
- Add
cursor: pointer;
to thebutton
, The cursor indicates to users there is an action that will be executed when clicking on it. - Use
<main>
instead of<div class="rating-container" role="main">
....
<main class="rating-container" > //... </main>
<div class="rating-container" role="main">
<--- this is a valid way to define a main role<main>
<--- but this is shorter and uses correct semantic HTMLI hope this is useful to you... Keep coding👍
1@developerwaleedPosted over 2 years ago@Bayoumi-dev Thank you soo much bro 🙏, I'll definitely apply the suggestions.
0 - Add
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