@danielmrz-dev
Posted
Hello @Renukta-21!
Your solution looks excellent!
I have just one suggestion:
- Use
main
to wrap the main content instead ofdiv
andarticle
.
š 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.
š And the tag div
has no semantic value. Since the main content goes inside the card, you can use the tag main
.
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!
Marked as helpful
@Renukta-21
Posted
@danielmrz-dev thanks buddy, looking forward to improve :)