Fylo Landing Page with two columns layout using CSS grid and flex

Community feedback

• 

  Lazar Nikolov•260

  @nikolovlazarPosted almost 4 years ago

  Iyanu this is just perfect! Everything's nicely executed, responsive and by the design. Great great great job! 🎉

  As for the <p> tag font, I notice that you're using the "Poppins" font, which is only imported at 600 weight. Instead, you should be using the "Open Sans" font, because it has the 400 weight you need. Check out the style-guide.md file under "Body, Call-to-actions". That piece of content is not a heading, so you should use the styling for "Body, Call-to-actions".

  This is so amazing! Keep up the great work Iyanu! 🙌

  1

  ---

  

  Iyanu Elijah Egbuwalo•145

  @enigmirePosted almost 4 years ago

  Thank you @lazarnikolov94.

  I will fix that.

  0

  

  Iyanu Elijah Egbuwalo•145

  @enigmirePosted almost 4 years ago

  Have fixed it <p> issue @lazarnikolov94. Truly, you never get lost with a second eye. Now left with the logo issue. I want to change the color to white.

  0

  

  Lazar Nikolov•260

  @nikolovlazarPosted almost 4 years ago

  @i-codde Emilie is right, just add filter: brightness(0) invert(1); in the .footer-logo class and you're good!

  0
• 

  Iyanu Elijah Egbuwalo•145

  @enigmirePosted almost 4 years ago

  Thanks @lazarnikolov94 @emestabillo.

  It worked

  0

  ---

  

  Lazar Nikolov•260

  @nikolovlazarPosted almost 4 years ago

  @i-codde you got it!

  0
• 

  P

  Emmilie Estabillo•5,540

  @emestabilloPosted almost 4 years ago

  Hi @i-codde, it looks like only 600 was uploaded for the <p> tag font, that's why it appears bold.

  For the logo, try this on filter brightness(0) invert(1). The combined properties should produce the white logo, or at least close to it.

  Another observation is that the sections use ids for styling. Better to use classes instead to keep specificity low, unless the section is being used as a scroll link.

  Hope this helps! And great job with this challenge!

  0

  ---

  

  Iyanu Elijah Egbuwalo•145

  @enigmirePosted almost 4 years ago

  Thank you @emestabillo.

  I will try that.

  0