Blog preview card using the cascade a bit

Hello there!

Congrats on completing the challenge! ✅

Your solution looks excellent!

I have just one suggestion:

• Use <main> to wrap the main content instead of <main> and <article>.

📌 The tag <article> would make more sense if the card was part of a bigger website (in certainly would in real world), but here it is all we have on the screen.

Unless you have a specific reason to use both, there's no need, considering that you can achieve the same result with only one. It also makes it easier positioning the elements inside.

This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.

I hope it helps!

Other than that, great job!

Hello there 👋. Congratulations on successfully completing the challenge! 🎉

• I have a suggestion regarding your code that I believe will be of great interest to you.

HTML 🏷️:

• This attribution element on your solution actually needs to be a footer element instead of div element which causes lacking of landmark for a webpage and allows accessibility issues to screen readers.

• What is meant by landmark ?, They used to define major sections of your page instead of relying on generic elements like <div> or <span>. They are use to provide a more precise detail of the structure of our webpage to the browser or screen readers

• For example:
  • The <footer> typically contains information about the author of the section, copyright data or links to related documents.

• So resolve the issue by replacing the <div class="attribution"> into a <footer> element in your index.html file to improve accessibility and organization of your page

• Additionally you need to move out the footer from being inside the main element, because the main and footer elements needs to be direct child of body element.

.

I hope you find this helpful 😄 Above all, the solution you submitted is great !

Happy coding!