Advice generator using React

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Awesome work with this project. Looking at your solution, I think you should know the following as well…

  • Inside the App component, this part could be wrap with main tag

  <div className="App">
        <AdviceContainer />
  </div>
  
  Into: 
  <main className="App">
        <AdviceContainer />
  </main>
  

  • Remove unnecessary code to keep it clean

  Besides that...

  • The design is well implemented
  • Proper use of components
  • Well structured React style
  • The style file is well organized
  • Proper hover state of the main button
  • JS functionality with random quotes generates well
  • Details of the design is on point

  ADVICE #76

  "You will always regret the round of Jägermeister."

  Above all, the design is well implemented. Keep up the good work!

  Marked as helpful

  1