@dewslyse
Posted
Hello @Cheng-KaWai! Congrats on completing this challenge. Your page looks good. Some things to note:
- You could wrap your content in a
main
tag so that your elements would be contained by a landmark. As such, you could change<section class="cards-container">
to<main class="cards-container">
. - It's nice that you try to use semantic elements in your markup and I think you've used the
article
tag well since the individual cards can be considered independent or self-contained. - Also, your page is missing an
h1
. Best practice dictates having on h1 element on a page for improved accessibility.
All the best