Advice generator app solution using Pure CSS

Community feedback

• 

  Elaine•11,360

  @elaineleung

  Posted almost 3 years ago

  Hi Ravindra, nice spinning animation on the dice! 😊 One suggestion I have is to add cursor: pointer on the dice icon so that users know it is interactive. Also, I'm viewing this on Firefox and for the first advice it took a while to load, and then the second one can't be loaded after I clicked on the dice. This is most likely due to Firefox's caching settings, and so if you want Firefox users to also use your app, you can try adding a header object in fetch, like this:

  const res = await fetch('https://api.adviceslip.com/advice', { cache: 'no-cache' });
  

  Lastly, for the mobile view, I think the component can use a bit more padding as the "advice number" line seems pretty close to the top of the component and the sides as well. Great work overall 😊

  10

  Marked as helpful
• 

  Ravindra Reddy•180

  @ravindra135

  Posted almost 3 years ago

  I appreciate your feedback, thank you so much.

  00