Riyana Gueco• 495
@rngueco
Posted
Hi Ivan!
Great job on this! 👍
There are a few minor details with regards to semantic HTML: heading level elements (h1, h2, h3, etc.) are usually used for topics and subtopics, so it does not make sense to use it for the statistics for this particular content since 10k+ or 314 or 1M does not really denote a topic or a subtopic. Also, it would be better if you put your statistics in a list, it makes semantic sense to do so.
But overall this was a fantastic first try! Cheers and happy coding! 🎉
Marked as helpful
1
Ivan Holguin• 170
@Ascecent
Posted
@rngueco Thank you so much for the feedback, i've updated the project with your suggestions :D
0
