Ahmed Bayoumi• 6,740
@Bayoumi-dev
Posted
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 HTML
I hope this is useful to you... Keep coding👍
1
Waleed Amjad• 80
@developerwaleed
Posted
@Bayoumi-dev Thank you soo much bro 🙏, I'll definitely apply the suggestions.
0