@mendezpvi
Posted
Excellent work! 👏
Here are some suggestions:
- Use semantic tags. The ´card´ can be an
article
orsection
. - In
published ...
for the date you can use thetime
tag. - The
avatar
class makes sense as adiv
. - When you use
article
orsection
it is mandatory that it has ah?
tag inside. You must remember that headings are consecutive. - Your CSS variables should be more declarative; for example,
w1
is just 2 letters that don't tell me anything; howeverfw-500
, makes some sense:font-weight: 500
. - The width of the card you are declaring with
vh
, you should tryvw
. I recommend you to checkclamp
, it can help you with the size of the card as well as the ‘font-sizes’.
I invite you to check my solution, you may find some ideas.
Happy coding! 😎
Marked as helpful