fylo-landing-page-with-two-column-layout-master
Design comparison
Solution retrospective
Thank you for any feedback guys!
Community feedback
- @ajlsdntPosted almost 3 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 helpful0@bahati7Posted almost 3 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 - @nonso01Posted almost 3 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 helpful0@bahati7Posted almost 3 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
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord