Artur Harutyunyan• 400
@ArturHarutyunyan1
Posted
Hello! looks great, just add a little JavaScript and it will be much better
Marked as helpful
1
Sugam Sahu• 100
@iSugam
Posted
@ArturHarutyunyan1 Thank you, and I've already added it :)
0
Design comparison
Solution retrospective
This is my 2nd solution to the frontend mentor challenge.
Any advice would be really helpful. Thank you :)
Community feedback
Deepak Joy Jose• 170
@deejayjay
Posted
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
@iSugam
Posted
@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
@deejayjay
Posted
@iSugam: All good. All the best!
1
Sugam Sahu• 100
@iSugam
Posted
@deejayjay All Done... Thanks again for the feedback.
1
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord