Planets fact using ReactJS

Community feedback

• 

  T

  Mike Hayden•1,005

  @mickygingerPosted over 3 years ago

  Hey Clément this looks great!

  The useEffect hook in React will call the callback method every time one of the dependencies updates, so for example:

  useEffect(() => {
    console.log(newImageSrc)
  }, [])
  

  In the code above newImageSrc will be logged to the console only once (on page load), and you may get a warning about missing dependencies.

  useEffect(() => {
    console.log(newImageSrc)
  }, [newImageSrc])
  

  In this example the newImageSrc will be logged every time it changes. By putting the variable in the dependency array you are telling useEffect that it is a dependency. Any variable or method that is used inside the useEffect callback is a dependency and should be included in the array.

  However you may not always want that (perhaps you only want the callback to be triggered when some of the dependencies change). There are several ways to handle that case outlined in this StackOverflow post

  Aside from that, it might be nice to fix the height of the main descriptive paragraph, so that the links don't move around as you are interacting with the site.

  I hope that's useful, and well done on the challenge! 🎉

  2

  ---

  

  Clément Creusat•1,655

  @ccreusatPosted over 3 years ago

  @mickyginger Thanks a lot ! I'm gonna look at this post and fix this. Same for the paragraph to avoid jumpy effect!

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  Congrats on being the first person to upload a attempt at this #GoodJob

  1

  ---

  

  Clément Creusat•1,655

  @ccreusatPosted over 3 years ago

  @palgramming Thanks man but I think I am the third person to upload it #private :)

  0