React Advice Generator

Community feedback

• 

  P

  Daniel•140

  @obriedanPosted over 2 years ago

  Hey Nathan,

  There should be the linebreak in the starter pack, it's in the assets folder.

  Is there a reason you have three <br /> tags? If you're trying to create space, consider margin or padding instead.

  Another small tip, you may want to destructure the props you send into your 'Advice.js' component.

  function Advice({number, quote}) {
  
  ..... rest of function 
  }
      
  

  Doing this means you only grab the props you need, and you don't need to use props.number or props.quote in your JSX.

  Marked as helpful

  0

  ---

  

  Nathan Yee•30

  @NathanYee412Posted over 2 years ago

  @obriedan Thank you for the tips! I've implemented every suggestion you had.

  0