Skip to content
Submitted over 4 years ago

HTML and FlexBox

GoharaJunior•125
@GoharaJunior
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

I took courage and faced a junior challenge because I have been studying programming for less than a year and it seems that I have done well. I'm open to tips where I can improve my code.

Community feedback

  • Gerben Dol•3,115
    @GerbenDolPosted over 4 years ago

    Hey Gohara! 👋🏻

    I agree, adding a mobile layout would really finish the job! It's looking really well on bigger devices! Maybe try looking into a mobile first approach? That way it will work on mobile by default and adjust to bigger screens with media queries.

    I'd also really like if you added some hover effects to your footer, but really, everything else is looking really good! 💪🏻

    1
    GoharaJunior•125
    @GoharaJuniorPosted over 4 years ago

    @GerbenDol Thanks for the Feedback, I am studying programming for less than a year, soon I will fix this project that I liked the result

    1
  • Gursimran•270
    @Gursimran07316Posted over 4 years ago

    hey gohara your solution work well for laptops and large display devices but it is not responsive it does not work well on mobo-devices i suggest u to work on this part too ;)

    1
    GoharaJunior•125
    @GoharaJuniorPosted over 4 years ago

    @Gursimran07316 Yes, soon I will focus on responsive and I will go back to this project and fix it, thanks for the Feedback.

    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