Advice Generator App

Community feedback

• 

  Troneci Andrei•260

  @TroneciAndreiPosted over 1 year ago

  Excellent work 👍 keep up and enjoy what you have learned!

  The layout seems to look good on all the screen size and I like how you structured the code and the fact that you use vite and not create-react-app that is a good thing

  The others already told you what you change I don't see anything else!

  1

  ---

  

  Harris N.•50

  @Harris170Posted over 1 year ago

  @TroneciAndrei Really appreciate your feedback. Thanks!

  0
• 

  Luka Glonti•3,440

  @lack21Posted over 1 year ago

  Excellent work 👍, but I have a suggestion!

  • Remove width: 100% from the .svelte-1ff3rsw, .svelte-1ff3rsw which is located in div is a block level element which by default takes full space available, so width: 100% does nothing here!

  1

  ---

  

  Harris N.•50

  @Harris170Posted over 1 year ago

  @lack21 Thankyou for pointing that out! I have removed it.

  0
• 

  Kamlesh Rajesh Yadav•5,060

  @Kamlesh0007Posted over 1 year ago

  Hey there! I noticed that your React app can take some time to load, and it might be a good idea to add a loader to improve the user experience.

  To add a loader in React, you can create a new component called Loader and use it to display a loading message or animation until your content has loaded. You can then conditionally render your Loader component based on the state of your app.

  0

  ---

  

  Harris N.•50

  @Harris170Posted over 1 year ago

  @Kamlesh0007 Thankyou for feedback. I will look into this.

  0

  

  Takesure Mamvota•0

  @MamvotaTakePosted over 1 year ago

  @Harris170 takesure

  0