@joaotfrodrigues
Posted
Hi Chelsea,
First of all, great job on your project! I loved the use of the CSS clamp
property, it adds a nice touch to the responsiveness of your design.
Now, the HTML semantic structure needs some changes. The card content should not be directly inside the <main>
tag, but instead wrapped in an <article>
tag within the <main>
.
Inside the <article>
tag, you could have a <header>
, <section>
, and <footer>
. The <header>
would be for the image, the <section>
for the content, and the <footer>
for the author details.
Regarding the hover animation, it's best practice to use the CSS pseudo-class :hover
for styling hover effects.
Overall, you’re doing a fantastic job! Keep up the great work and consider these adjustments to further enhance your project.
Best regards, João Rodrigues