Riyana Gueco• 495
@rngueco
Posted
Hi Ricardo!
This looks good!
You could make your HTML more semantic by using semantic elements. For example, it does not really make sense to have a heading of "10k+" since it does not denote a topic so it typically should not be wrapped in a heading element (<h2>). You could also put your stats in a list since it makes semantic sense to have them in a list 🙂
Good job on the layout! 👍
Happy coding! 🎉
Marked as helpful
0
@Levarick
Posted
@rngueco
Hi Riyana! Thank you!
You are right! I would use a <ul> to make a list and not using a h2. I've to be better at semantic (and maybe my english). Thanks a lot for the comment!
0
