Skip to content
Submitted over 3 years ago

Responsive landing page using CSS Flexbox and Grid

Mohamed•325
@MohamedBehhar
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Fylo dark theme landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Hello, frontend mentor community, This challenge was a bit difficult for me, especially the footer section with a lot of divs, I'm happy with the result but I'll definitely try to make my code as dry as possible in the future. I'll be grateful if someone could review my code and give me some advice.

Community feedback

  • Suraj Singh•0
    @suraj-singh127Posted over 3 years ago

    Brother this is awesome work! You must have worked really hard on this and I can assure it has paid off. Keep Going!!!

    0
    Mohamed•325
    @MohamedBehharPosted over 3 years ago

    @suraj-singh127 Thanks for the reply, to be honest, is kind of challenging a little bit, but I'm not 100%satisfied with the result.

    0
  • Patrick•14,265
    @palgrammingPosted over 3 years ago

    you should put your <li> in a <ul> or <ol> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/li

    0
    Mohamed•325
    @MohamedBehharPosted over 3 years ago

    Thanks again, i didn't pay attention

    0

Please log in to post a comment

Log in with GitHub

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