@BenjaminSemah
Posted
Hi Mbungai. Great job done! 👍
HTML: For your HTML, I will encourage you to consider using semantic HTML elements. Divs are useful, but for the sake of accessibility, SEO, and improving code readability, using these semantic elements like main, section, footer, button, ul and li (for list items) is highly recommended.
CSS: Your CSS looks good! The only feedback I will give is on where you use blocks:nth-child
.
For example, you do this for all 4 blocks.
.blocks:nth-child(1) {
background-color: #FFF6F5;
}
.blocks:nth-child(1) {
color: hsl(0, 100%, 67%)
}
Since it’s targeting the same element. You can do the following instead. This does the same thing and it’s more concise. And same for the other three .block
elements too.
.blocks:nth-child(1) {
background-color: #FFF6F5;
color: hsl(0, 100%, 67%);
}
Cheers and happy coding!
Marked as helpful
@Mbungai-Francesco
Posted
Thanks for the proposals Sir, I'll do more research on semantic HTML elements and look at integrating them in my coding journey @BenjaminSemah