Skip to content
Submitted over 4 years ago

Fylo landing page - flexbox only

Michal Gregulec•160
@michalgregu
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Fylo landing page with two column layout
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Solution retrospective

Looking back, i would definitely use grid instead of only flexbox, wanted to have some practice with flex though. Thanks for looking!

Community feedback

  • Edgar Burtnieks•465
    @edburtnieks-privatePosted over 4 years ago

    Hey @Michal!

    You solution looks good :) I think it's great that you chose to use Flexbox in most parts. I actually would have used Flexbox too :)

    Just quickly looking through. I noticed you have justify-content: flex-end; on header. You don't need it if you have margin-right: auto; to logo :) Or you could remove margin-right from logo and leave justify-content.

    Nice work, keep it up :)

    1
    Michal Gregulec•160
    @michalgreguPosted over 4 years ago

    @edburtnieks hey, i followed your advice and you're absolutely right, i don't need the flex-end property. thanks for the tip!

    0
    Edgar Burtnieks•465
    @edburtnieks-privatePosted over 4 years ago

    @michalgregu You're welcome :)

    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