Blog Card Preview using html and css basic

Community feedback

• 

  Marcos Travaglini•4,840

  @BlackpachamamePosted 10 months ago

  • You should try placing the CSS in a separate file and then import it into the HTML
  • The font does not work, because you are not importing it from anywhere, the README of the styles should give you a link to select the font from Google Fonts and the link to import it.
  • I have seen that you have used br to apply the space above and below, it is the first time I see it and I was surprised, I did not expect it haha. Anyway, it's not the right way to do it. The correct thing to do would be to remove those br, remove the margin from your pseudo id and apply a padding: 20px. This will apply that spacing around the entire card.
  • I recommend using class instead of id

  The rest looks really good, keep it up! 💪

  Marked as helpful

  1

  ---

  

  ChaosBlitz404•140

  @ChaosBlitz404Posted 10 months ago

  Thx for the feedback. I appreciate it @Blackpachamame

  1
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 10 months ago

  Hello @ChaosBlitz404!

  Your solution looks great!

  I have one suggestion:

  • I noticed that you set display: grid; on the body to center the card, but there are 3 rows since you added the <br> (break row) tags. This is pushing your card up.

  You can fix that by adding this to the body:

      grid-template-rows: 90% 1fr;
  

  Or you can just remove the <br> tags. 😊

  I hope it helps!

  Oyher than that, you did an excellent job!

  0