@correlucas
Posted
πΎHello Leonardo, congratulations for your new solution!
Your component card its really good done. But you can do some work around the html
structure in order to have a cleaner code. For example, you can replace the div
with the <main>
tag. Remove all the other divs and wrap everything just with a single block, in this case the main
.
To clean the css you can remove all the classes and manage everything just by using the direct selector for each element (main, div, h1, img and p).
To have a better perfomance with the image scaling use ever display: block
and max-width: 100%
to have the image scaling with the container size.
π I hope this helps you and happy coding!
Marked as helpful
@LeonardoSchuquel
Posted
@correlucas @correlucas Hi @correlucas thanks for the feedback. It was very useful for me and I have already applied the changes