blog preview card

Community feedback

• 

  Asv000•190

  @Asv000Posted 6 months ago

  From my point of view besides of it's height on preview project looks pretty fine. I think semantic HTML is fine but I don't like that you used button for Learning. I don't really thing that's neccessary and you could achieve that in other way eg. using P with display: inline-block and padding. You definetly need to practice with RWD but don't worry MDN got good guide for why and when. I'm also learning a new things brother, we all gonna make it :) I see that your CSS file looks kinda weird, but pretty similar to SCSS Check this SASS introduction, maybe you'll like it! Remember to never give up and keep trying!

  Marked as helpful

  0
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 6 months ago

  Hello there!

  Congrats on completing the challenge! ✅

  Your project is looking fantastic!

  I'd like to suggest a way to make it even better:

  • Using margin isn't always the most effective method for centering an element.

  Here's a highly efficient approach to position an element at the center of the page both vertically and horizontally:

  📌 Apply this CSS to the body (avoid using position or margins in order to work correctly):

  body {
      min-height: 100vh;
      display: flex; 
      justify-content: center;
      align-items: center;
  }
  

  I hope you find this helpful!

  Keep up the excellent work!

  1