romila• 3,570
@romila2003
Posted
Hi Jeremy,
Congratulations for 🎉 for completing this challenge, the Chart component looks great, and I like that you used Char.js, I used it too. There are some issues/suggestions want to address:
- Even though, you used the correct semantic for the header and the main content, you should also wrap the footer within the
footertag e.g.<footer class="attribution"></footer> - Your HTML is missing the
langattribute e.g.<html lang="en"></html> - Since you used the
sectiontag, you should include a header, anywhere betweenh2toh6.
Overall, great work and wish you the best for your future projects so keep coding 👍.
0
