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

landing page with BEM and sass

tortaruga 450

@tortaruga

Desktop design screenshot for the Loopstudios landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

it looks good enough, but next time i'd try making it look better on medium sized screens and I just noticed the hero image should be darker, i forgot to add that

What challenges did you encounter, and how did you overcome them?

had some issues setting up sass and then with the layout as always, but after trying fifteen thousand times everything worked.

also, on my screen it doesnt look like in the design comparison, not sure what's up about that

What specific areas of your project would you like help with?

all feedback's welcome

Community feedback

@jjdavenport

Posted

The basic layout of the site is correct, on the desktop layout the large areas of blank space are likely caused by min-height: 100vh. However the mobile layout is spot on. The hover effects are not quite identical to the solution with the borders on hover being 100% instead of 50% of the element width, I also have a wierd bug on 1440p where I can only see half of the h1. The page could be made more accesible with semantic html elements used on the mobile menu, I would also disable scroll when the mobile menu is active with js. The desktop site needs more work to make is responsive on the viewports its designed for.

Marked as helpful

0

tortaruga 450

@tortaruga

Posted

Thank you @jjdavenport, that was really helpful! For the hover effect being 100% of the width it was because I thought the solution meant it had to 'expand' from the center when you hover it and gradually underline the whole word, but I think I just made it unnecessarily more complicated 😅

1

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