Submitted over 5 years ago

Fylo landing page using Flexbox

@Senatrius

A solution to the Fylo landing page with two column layout challenge

Solution retrospective

This definitely felt a fair bit tougher than the previous ones I did. On top of that, I took a break in the middle of this project and the code got out of hand really fast when I came back. Any comments/advice/tips on general code quality (and inevitable redundancy I'm sure) are always appreciated :)

Code

Please log in to post a comment

Community feedback

Adarsh Pratap•5,515
@adarshcodes
Posted over 5 years ago
Hey! @Senatrius, you did great work on this project, it looks amazing👍 Let's see other things,

1.Design
- your design looks great, hover-effect, layout, positioning of images everything works fine.
1. Responsiveness
- Responsiveness also works nicely.
1. Code
- Code is well-structured
1. Issues aroused
- Solve the HTML issue
Everything is nice, keep coding🐱‍💻.
Vincenzo Marcovecchio•1,195
@VincenzoMarcovecchio
Posted over 5 years ago
Good job man!

From what I can notice here maybe try to fix that <nav></nav> element, give it some align-self so that when you hover on the link it's not jumping too much

and try to make the telephone number and the email address on the footer links you can click

Keep practicing 👍 :)
Matt Studdert•13,611
@mattstuddert
Posted over 5 years ago
Awesome work on this challenge, Senatrius! Your code looks great! There's not much I'd change with your HTML and your Sass code is well-written and is generating clean CSS selectors with low specificity.

Also, it's been great seeing you give feedback to others. You've been giving some great advice 🙂

Keep up the great work!

Join 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