Fylo landing page with two column layout - Mobile first

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋Hi Shibu Sarkar!

  First thing you need to know is that, I opened this solution on my mobile phone, which meant everything that I said was based on what I saw on my mobile device (360px width).

  I have some feedback for this solution:

  • The page is not 100% width, it's showing a white color on the right side.
  • The header logo and the nav links are too small and the they are side by side.
  • You should have email validation, you can just use CSS :invalid for that (no need JavaScript).
  • The See how Fylo Works link color is blue, while it should green color from the style-guide.md.
  • The footer text is too big, try to make it smaller.
  • The social icons on footer is too small.
  • For the logo alt attribute, you cam just say the company name Fylo, no need the logo.
  • For the useless or meaningless images, you should just leave the alt="" attribute empty.

  That's it! Hopefully this is helpful!

  Marked as helpful

  0

  ---

  

  Shibu Sarkar•605

  @shibuwdPosted over 3 years ago

  @vanzasetia Thank you so much for your great suggestion. I will fix it ASAP.

  0
• 

  Amon•2,560

  @A-amonPosted over 3 years ago

  Hello! Great work~ About the overflowing picture tag, try using transform:scaleY instead of scale so that it only scale vertically. And I think the footer might need some padding and some adjustments to the font size.

  Marked as helpful

  0

  ---

  

  Shibu Sarkar•605

  @shibuwdPosted over 3 years ago

  @A-amon Thank you so much for the suggestion.

  0

  

  Amon•2,560

  @A-amonPosted over 3 years ago

  @Shibu77 Glad to help!

  0