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

Clipboard landing page (Html,Scss & Animations)

@sauravchamoli17

Desktop design screenshot for the Clipboard landing page coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


It would mean a lot to me if you could check my solution out and review it.

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Awesome work, Saurav! Your solution looks great! Here are a few small pointers after taking a look at your code:

  • Your HTML looks great. The only thing I'd add would be more semantic HTML5 elements like header and section. Also, a quick heads up that the footer is currently outside of the body, which shouldn't be the case.
  • In your CSS you've got this media query: @media (min-width: 320px) and (max-width: 480px). You could just remove the (min-width: 320px) condition as that would take care of everything below 480px.
  • Nesting is a really useful feature of Sass, but be careful not to overuse it as it can lead to overly specific CSS selectors. For example, in your generated CSS you've got a selector of footer .footer_content .footer_links .links which could just be .links.

I hope these tips help. Keep up the great work!

1

@sauravchamoli17

Posted

@mattstuddert Thank you very much Matt! I was waiting for your review on the solution and your tips always helps me to become a better developer!

0
P
Matt Studdert 13,611

@mattstuddert

Posted

@sauravchamoli17 you’re welcome!

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