Design comparison
SolutionDesign
Community feedback
- @AFuad-techPosted 8 months ago
Good work but here are a few tips for improvements:
- You didn't needed to declare container class if you are not going to use it
<div class="container"><div class="owncard">
- You added eventlistner twice you could have added once.
removeclass(list); ulist.classList.add("active"); let rating = ulist.innerText; document.getElementById("button").addEventListener("click", () => {
- And Adding innerhtml is a bit redundant as it is better to add it in HTML and just set the display to none.
0 - You didn't needed to declare container class if you are not going to use it
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