All comments

• 

  sai•130

  @Kondurusai2903

  Submitted over 1 year ago

  

  interactive-rating-component-main

  1

  10
  

  AFuad-tech•60

  @AFuad-tech

  Posted over 1 year 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
• 

  P

  Ed•720

  @EdwinSch

  Submitted over 1 year ago

  

  FAQ accordion with React

  #react

  1

  30
  

  AFuad-tech•60

  @AFuad-tech

  Posted over 1 year ago

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

  1