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

Pure HTML & CSS , Flex box

@Mazen922

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


Hi Frontends

I need your feedback , I just finished my Challenge 😊.

Community feedback

@Ximoromero6

Posted

Hi Mazen,

I just checked your challange and I can say you did a great job, however let me give you some little datails to improve it a bit:

  • You could use the property min-height: 100vh on the body element and use flexbox to center vertically your container.
  • Also you can use the property line-height: 22px (you can play with the measures) to give some space to your paragraph.
  • The width of the border on the container should be 1px to make it closer to the given design.
  • And last, you could give a margin bottom on the part of the name.

That's all! Have a good day Mazen :)

Marked as helpful

0

@Mazen922

Posted

I will apply it in my code 😊@Ximoromero6

Thanks for your time.

1

@Bishalsnghd07

Posted

Hi, @Mazen922👋

Amazing work🎉

Just one suggestion for you regarding css basics, you need to add min-height:100vh which is missing in your main landmark to center your div and give some padding on it to look more responsive in mobile view also.

Hope, this suggestion will help you!

Happy Coding❤️

Marked as helpful

0

@Mazen922

Posted

Amazing @Bishalsnghd07

I will do it , Thanks 😊

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