Blog Preview Card

Community feedback

• 

  João Vitor Morita•210

  @joaomoritaPosted 6 months ago

  Hey! Great work there! You could use more semantics HTML. You didn't need a container div if you already have a main, you could put the main css at the body tag, and the container css at the main tag. You can use more responsive units to your image and paragraphs too. It's very helpfull look how other people did the same challenge and look for ways to improve. Check out this article about semantc elements: https://www.w3schools.com/html/html5_semantic_elements.asp

  I hope I can help you in some way

  1

  ---

  

  KingX•150

  @iwedibahPosted 6 months ago

  @joaomorita Thank you so much for the review and your correction i will effect it in my next challenge

  0
• 

  gabriel 🍉•70

  @eugabrieldesousaPosted 6 months ago

  s

  1

  ---

  

  KingX•150

  @iwedibahPosted 6 months ago

  @eugabrieldesousa thank you gabby tho i am trying to understand the comment

  0

  

  KingX•150

  @iwedibahPosted 6 months ago

  @eugabrieldesousa thank you gabby tho i am trying to understand the comment

  0

  

  KingX•150

  @iwedibahPosted 6 months ago

  @eugabrieldesousa thank you gabby tho i am trying to understand the comment

  0

  

  gabriel 🍉•70

  @eugabrieldesousaPosted 6 months ago

  @iwedibah oh sorry man, its a wrong comment for my part, now, your code is good, maybe your padding is more than necessary

  0

  

  KingX•150

  @iwedibahPosted 6 months ago

  @eugabrieldesousa alright gabby i sincerely appreciate

  0