Skip to content
OverviewSolutions
7
Comments
2

Atef Zaky

@AtefZakyEgypt180 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

View more solutions

Latest comments

  • Atef Zaky•180
    @AtefZakyPosted over 1 year ago

    Hello! congratulation on completing your first challenge 🤟

    I have some suggestions to make it better and I will try to answer your questions🔽

    • in the mobile view you should give some padding to the container to make space between the card and the browser window in extra small devices. and change the height: 100vh to min-height: 100vh like that when the device's height is small your content will take up its space and the scroller will appear

    • in this challenge, you don't need to give the container max-width because the content doesn't take the full screen but when you make this you will need to give the background color to the body or the parent of the container to see the background on a large screen.

    I hope this helps you, overall nice solution, happy coding 👨‍💻

    Marked as helpful
    1
  • Atef Zaky•180
    @AtefZakyPosted over 1 year ago

    Good job.., but you need more padding in the footer and the header, Increase the size of the logo in the header and footer and the size of links in the header, Make the heading in the first section and the testimonials section uppercase and you can give the first section min-height: 100vh like that when the device screen height is small your content will remain on its shape Good luck.

    Marked as helpful
    0