Completed with HTML5, CSS3, Grid, FlexBox, JavaScript, and API

Community feedback

• 

  Artur Harutyunyan•400

  @ArturHarutyunyan1Posted over 2 years ago

  Hello! looks great, just add a little JavaScript and it will be much better

  Marked as helpful

  1

  ---

  

  Sugam Sahu•100

  @iSugamPosted over 2 years ago

  @ArturHarutyunyan1 Thank you, and I've already added it :)

  0
• 

  Deepak Joy Jose•170

  @deejayjayPosted over 2 years ago

  Good job on making the page look so close to the mock up. My advice is to focus on making your HTML semantic. Also, I noticed that you have hard coded the advice instead of loading the advice from the Advice Slip API. If you are new to JavaScript, this can easily be implemented referring the example in this page: "https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API/Using_Fetch". If you have any questions, feel free to ask via slack. Good luck!

  Marked as helpful

  1

  ---

  

  Sugam Sahu•100

  @iSugamPosted over 2 years ago

  @deejayjay Thank you for replying. Actually, I didn't know that I had to fetch an API, I thought it was hard-coded 😅. Really thank you for the feedback. I will update it soon.

  1

  

  Deepak Joy Jose•170

  @deejayjayPosted over 2 years ago

  @iSugam: All good. All the best!

  1

  

  Sugam Sahu•100

  @iSugamPosted over 2 years ago

  @deejayjay All Done... Thanks again for the feedback.

  1