@correlucas
Posted
๐พHello Karishma, congratulations for your new solution!
Youโve done really good work here putting everything together, Iโve some suggestions you can consider applying to your code:
Your html is working but you can improve it using meaningful tags and replace the divs, for example the main div tht takes all the content can be wraped with <main>
or section, about the cards you can replace the <div>
that wraps each card with <article>
you can wrap the paragraph with the quote with the tag <blockquote>
this way you'll wrap each block of element with the best tag in this situation. Pay attention that <div>
is only a block element without meaning.
Here's a complete guide for HTML semantic TAGS: https://www.w3schools.com/TAgs/default.asp
Use a CSS reset to avoid all the problems you can have with the default CSS setup, removing all margins, making the images easier to work, see the article below where you can copy and paste this css code cheatsheet: https://piccalil.li/blog/a-modern-css-reset/
โ๏ธ I hope this helps you and happy coding!
Marked as helpful
@Kdt98
Posted
@correlucas Hi Lucas, thanks for sharing feedback and helpful resources.