Advice Generator

Community feedback

• 

  Loo_cee•230

  @LooceePosted about 2 years ago

  Hi, maybe you could align the texts in the body to center

  body { text align: center; }

  I think you should also remove the height property from your container so it can adjust to changes

  Add margin top and bottom to the content in your container excluding the button, so they can be evenly spaced.

  Rather than leaving your page empty till the user clicks the button, I think you should add some content like Advice #117 as given in the design files.

  I hope this helps and happy coding.

  Marked as helpful

  0

  ---

  

  Raphael Alves Ferreira•70

  @rapha0311Posted about 2 years ago

  @Loocee thank you for the tips! It's always nice to receive constructive criticism.

  0