Design comparison
Community feedback
- @Kamlesh0007Posted over 1 year ago
Congratulations on completing the challenge!🎉. That's a great achievement, and I'm sure you put a lot of effort into it. I really liked the way you approached the challenge and the code you wrote. You demonstrated a good understanding of the concepts and applied them effectively to solve the problem.I have a few suggestions to improve your code further. I noticed that your app can take some time to load, and it might be a good idea to add a loader to improve the user experience. You can add a
loader
while fetching data from an API, you can follow these steps: add this code in ur javascript with css of loader in ur stylesheetconst loader = document.getElementById('loader'); function showLoader() { loader.style.display = 'block'; } function hideLoader() { loader.style.display = 'none'; } // Example of fetching data from an API button.addEventListener('click', () => { showLoader(); axios.get('https://api.adviceslip.com/advice') .then((response) => { hideLoader() quote.innerHTML = response.data.slip.advice id.innerHTML = `advice #${response.data.slip.id}` }) })
Marked as helpful0@JeremiasRincaveskiPosted over 1 year agoThank you very much for your suggestion, I will try to implemente @Kamlesh0007
0
Please log in to post a comment
Log in with GitHubJoin 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