fylo-landing-page-with-two-column-layout-master

Community feedback

• 

  iajlsdnt•70

  @ajlsdntPosted over 2 years ago

  Hi Bahati it's Amelia (@ajlsdnt) from twitter :D This looks awesome, great job. It's so cool you use 'hamburger' menu for the mobile media query.

  I may not give many comments on the code (I haven't touched any framework myself), but here you go:

  • you can wrap your <nav> with <header> tag
  • try to use 'box-shadow' for the '.card (the testimony box) instead of 'border'
  • background transparent for the 'see how fylo works link'
  • and background white for the first section 'showcase'

  Cannot wait for your next projects of FrontEnd Mentor :)

  Marked as helpful

  0

  ---

  

  Philemon Bahati•580

  @bahati7Posted over 2 years ago

  @ameliajulis hey Amelia! Thanks for appreciating my work. I'm glad you think it's cool. Your advices has improved the look of the site and I think you have a good design quality. Please take another look at it.

  0
• 

  Martin•460

  @nonso01Posted over 2 years ago

  well! looks pretty good! but you know the goal of completing challenges is to test your css skills not frameworks! anyway still a good solution 🙂

  Marked as helpful

  0

  ---

  

  Philemon Bahati•580

  @bahati7Posted over 2 years ago

  @nonso01 Thanks bro you're right. This was my first time to use Bootstrap and I was so exited to work with it but anyway the next project I'll work with only Html and Css. Thanks

  0