Fylo landing page with two column layout

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Hey, great work on this one. In terms of layout, the desktop is fine, just some tweaking of padding will be awesome. The mobile is good and no issues when resizing the layout.

  Suggestions would be that:

  1. You navbar links should be wrapped inside a tags. Right now, they are just inside of li which doesn't do anything. Inside that li wrapped the text inside their respective a tags. We need to make our markup clean and semantic.

  2. Font sizes should be adjusted as well, since there are sections where the font-size are really small. Like the testimonial the text are small and hard to read. Tweaking those will be really good and the testimonial itself, the background-color is different, as well the whole section. You forgot to change the background-color of it. Though I think you did not include it, since if you include it, it will overlap the curve image right. That curved image could just been inside ::before or ::after, that way you can copy like the original.

  3. Footer links should be as well wrapped inside their a tags. Remember to put all supposed links, inside a tags, including the social media icons.

  But still, great job on this^

  0

  ---

  

  Isenraf•220

  @IsenrafPosted over 3 years ago

  Hi @pikamart thanks for the great feedback I completly forgot that i had to do things in a semantic and clean markup(will fix this as well). Thanks again

  0
• 

  Michal•665

  @mbart13Posted over 3 years ago

  hi,

  you need to put "margin: 0 auto" on the body

  currently content is not centered

  0

  ---

  

  Isenraf•220

  @IsenrafPosted over 3 years ago

  ok @mbart13 will fix that. Thanks

  0