Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Blog preview card built using simple HTML and CSS

@Nsebo

Desktop design screenshot for the Blog preview card coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello Everyone, The challenges I faced building this preview card was the media query and width of the container.

All feedbacks are welcome. Thank you!

Community feedback

Marcos Travagliniโ€ข 4,920

@Blackpachamame

Posted

Greetings! you have done a great job ๐Ÿ˜Ž

๐Ÿ“Œ Some accessibility and semantics recommendations for your HTML

  • To improve the semantics of your HTML, you can change your <div class="container"> to a <main class="container"> and the <div class="attribution"> to a <footer class="attribution">

๐Ÿ“Œ Some suggestions

  • You can apply display: block to the image to remove the white space generated underneath. Although visually in this case it is irrelevant, it helps you better calculate the space with other elements
  • Apply max-width: 100% to your img so that it occupies the correct width within the container. Remove width: 300px
  • Instead of using margin to center your content in the center of the screen, you can use the flexbox properties in the body:
body {
      font-family: 'Figtree', sans-serif;
      background-color: var(--background-color);
      padding: 30px 40px;
      min-height: 100vh;
      display: flex;
      justify-content: center;
      align-items: center;
      flex-direction: column;
      gap: 20px; /* Separate the main from the footer */
}

Marked as helpful

2

@Nsebo

Posted

@Blackpachamame Thanks so much for your honest feedback. Your code made more sense when I added it to mine :) I had problem understanding the difference between max-width and width, I need to learn more on max-width and width. Thanks once again for taking your time to correct my work.

1

@MelvinAguilar

Posted

Hello there ๐Ÿ‘‹. Good job on completing the challenge !

I have other suggestions about your code that might interest you.

  • The cursor: pointer property should only be used on interactive elements that perform an action. If you apply it to a <main> tag, you might confuse the user, making them think that clicking will trigger something.
  • To make the alt attribute as useful and effective as possible, avoid using words such as "image", "photo", or "picture" as they are redundant because the image tag already conveys that information.
  • For a photo of a person, use their name as the alt text
  • Instead of using pixels in font-size, use relative units like em or rem. The font-size in absolute units like pixels does not scale with the user's browser settings. Resource ๐Ÿ“˜.

I hope you find it useful! ๐Ÿ˜„

Happy coding!

Marked as helpful

1

@Nsebo

Posted

@MelvinAguilar Thanks for your honest feedback and your suggestions, your suggestion made me noticed that I was done with coding. I will have to add the pseudo class property :active and color changes when clicked on the title.

Thanks for the resource added, will read more the on the em and rem. Thank you!

0
Daniel ๐Ÿ›ธโ€ข 36,160

@danielmrz-dev

Posted

Hello @Nsebo!

Your project looks great!

I noticed that you used margin to place the card in the middle of the page. Here's a very efficient way to center the card:

  • Apply this to the body (in order to work properly, don't use position or margins):
body {
    min-height: 100vh;
    display: flex;
    justify-content: center;
    align-items: center;
}

I hope it helps!

Other than that, great job!

Marked as helpful

1

@Nsebo

Posted

Thank you for taking your time to look through my work, I appreciate your honest feedback and i will do the needful to my code. Thank you! @danielmrz-dev

1
Abdulโ€ข 8,560

@Samadeen

Posted

Hello Nsebo

Great work on completing this challenge. Cheers!! ๐ŸŽŠ๐ŸŽŠ

Some adjustments are needed to place your container in the middle properly.

    display: grid;
     place-content: center;
     min-height: 100vh; 

I hope to see more solutions from you.

1

@Nsebo

Posted

Thanks for your honest feedback, I will make the correct right away. Thank you!@Samadeen

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord