@danielmrz-dev
Posted
Hello @evengene!
Your solution looks excelent!
I have just one suggestion:
- In order to make your HTML code more semantic, use
<h1>for the title instead of just the tag<a>. The text HTML & CSS Foundations is not just a link, but also the main title of the screen.
š You don't need to replace it, just include the <h1> inside the tag <a> around the text.
This change has little visual effect on the project, but it makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, you did a great job!
@evengene
Posted
@danielmrz-dev thank you for providing feedback,
I didn't consider adding the blog post title as an <h1> because in my mind it's a part of a larger page with real header, titles and etc., so this blog card is lower in hierarchy than other potential elements.
If anything I would have the blog title to be h4 or similar depending on the complexity of a blog page.
