Skip to content
Submitted over 2 years ago

Responsive landing page using SCSS

Marcel Qayoom-Taylor•30
@Marcelqayoomtaylor135
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I found managing the margins difficult as adjusting them seemed to impact other elements when I didn't intend for that to happen.

Flexbox. While it works, I'm unsure if grid would have been better here.

How do you manage margins in the best way?

Community feedback

  • Raj Prasad Sanjel•980
    @RajSanjelPosted over 2 years ago

    Congrats on completing 1st challange:-

    • You can give padding instead of margin(margin is outside)(padding is inside).
    • Improve mobile version the design is touching the edge. (you can give margin-top and bottom).
    • Use semantic html read about it by clicking here

    Hope this helped.

    Marked as helpful
    1
    Marcel Qayoom-Taylor•30
    @Marcelqayoomtaylor135Posted over 2 years ago

    @RajSanjel Great feedback! Thanks Raj

    0
    Raj Prasad Sanjel•980
    @RajSanjelPosted over 2 years ago

    @Marcelqayoomtaylor135 You are welcome.

    0

Please log in to post a comment

Log in with GitHub

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