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

Fylo Landing page

@SuriyanRavi

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

Community feedback

Tharun Raj 1,250

@Code-Beaker

Posted

Hi there, congratulations on completing this project. You've done a great job with this one!

I have some small suggestions regarding your website that might help you make it better:

  • Use class instead of id in your HTML.
  • I attempted to visit your website on a desktop and I noticed that, the <img id="img2" src="./images/illustration-2.svg" alt="" width="500px"/> was overflowing my screen (image from "Stay productive, wherever you are" section).

This can be prevented by removing the width="500px" from your html and adding reset to your css:

img {
max-width: 100%;
}
  • instead of flex, for a better and easier layout control, you can your display: grid for your sections. Like this:
#content2 {
display: grid;
grid-template-columns: repeat(2, 1fr);
}

this will be better for handing 2-column layout compared to flex.

Note: change the id selector to either a class or to your liking...

  • The footer links have poor contrast making it hard to see and read. Try a lighter color for them.

  • Mobile responsiveness of the site still needs some work. Currently, it looks like the width of the whole body is shrinking and not the elements.

I hope you find this helpful... 😊

0

@SuriyanRavi

Posted

@Code-Beaker Thank you for your suggestion. I Seriously dont know what the error in responsive width. I tired much to solve it but i couldn't. If you know help guide and help me with responsiveness.

0
Tharun Raj 1,250

@Code-Beaker

Posted

Hi, @SuriyanRavi

I'm sorry about that, it looks like the responsiveness is now fixed or it was an error from my side. You've done well with the responsiveness and now it looks good.

But there are still some things that need some work that I have mentioned in my original comment.

Good luck! 👍

Marked as helpful

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