Skip to content
OverviewSolutions
2
Comments
2
AFuad-tech
@AFuad-tech

All comments

  • AFuad-tech•20
    @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
  • AFuad-tech•20
    @AFuad-techPosted 10 months ago

    Good work, especially with the use of react. Nothing to add really. Keep up the good work.

    1