Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Fylo Landing Dark - SCSS and JS

P
Daveā€¢ 5,245

@dwhenson

Desktop design screenshot for the Fylo dark theme landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


This is my second challenge trying out SCSS. I enjoyed SCSS much more this time around as I am getting more comfortable with it.

Any comments on how I can improve my use of SCSS would be particularly appreciated. Any other comments or feedback on any points are also very welcome.

PS. I bumped up the font size compared to the design specification as I found it to be very small in the design....

Community feedback

P
ApplePieGiraffeā€¢ 30,545

@ApplePieGiraffe

Posted

Hello, Dave! šŸ‘‹

Nice job on another challenge! šŸ˜€

I suggest decreasing the space between the top of the page and the header (since there seems to be quite a bit of margin-top on the header, especially on smaller screens). One other detail that I think would be a nice touch is to allow the background of the footer to extend to both sides of the screen so that there isn't any empty to the sides of the part of the design on larger screens. šŸ˜‰

Those are just small things, though. Overall, your solution looks pretty great and responds nicely! šŸ‘

Keep coding (and happy coding, too)! šŸ˜

1

P
Daveā€¢ 5,245

@dwhenson

Posted

@ApplePieGiraffe Thanks again! Yes, I think you had mentioned that footer issue on another challenge. I'll try and remember that in future.

I'm always a but unsure with these challenges whether things with a background colour should spread out beyond 1440px, or be constrained. Luckily, it's an easy fix!

Cheers.

0
P
Daveā€¢ 5,245

@dwhenson

Posted

Oh please ignore that extra-wide button in the screenshot - it's due to me using the fit-content value which isn't recognised in Firefox and maybe some other browsers.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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