Todo app with React

Community feedback

• 

  Fazza Razaq Amiarso•2,340

  @fazzaamiarsoPosted over 2 years ago

  Hello Nika! Great solution and clean code overall!

  Here is a quick tip. You should always have a clean-up function for the listener.

  useEffect(()=> {
  const resizeFn = () =>  setWidth(window.innerWidth)
    window.addEventListener("resize", resizeFn)
  return () => window.removeEventListener("resize", resizeFn)
  }, [])
  

  I hope it's useful! Cheers!

  Marked as helpful

  1

  ---

  

  nika-sergeeva•150

  @nika-sergeevaPosted over 2 years ago

  @fazzaamiarso Thanks a lot! I forgot to write it. =)

  0