Fylo Landing page

Community feedback

• 

  Tharun Raj•1,330

  @Code-BeakerPosted 7 months ago

  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•180

  @SuriyanRaviPosted 7 months ago

  @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,330

  @Code-BeakerPosted 7 months ago

  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