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

sass, flexbox and grid.

Chrisβ€’ 740

@chrisbailey85

Desktop design screenshot for the Fylo landing page with two column layout coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback will be appreciated thank you.

Community feedback

Roc Tanweerβ€’ 2,500

@RocTanweer

Posted

Hello πŸ‘‹ @Chris

Adding to what @pikamart said

Your layout is quite far from the design...

Use irfan view to measure the layout in the design image they provide or else have a premium membership, whatever you wish for

Hope it helps ☺️ now you can have both functionality and design and not just the former one

Happy coding πŸ˜‰

1

Chrisβ€’ 740

@chrisbailey85

Posted

@RocTanweer thank you.

0
Raymart Pamplonaβ€’ 16,140

@pikapikamart

Posted

Hey, great work on this one. The layout is good in both desktop and mobile.

Everything seems resizes well when going in mobile as well.

But I think that you are using the wrong font in your footer fag. Right now it is using times new roman which is default. Might want to look on that one.

Also maybe add a padding in your header to the left and right side in mobile state. So that the brand logo won't be touching the sides of the viewport.

Lastly, your facebook icon at the footer, it is different in terms of shape with its other sibling.

Overall, you did a really good job^

1

Chrisβ€’ 740

@chrisbailey85

Posted

@pikamart thank you. I'll update that now.

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