Skip to content
Submitted over 4 years ago

Position, flex, Sass (scss)

M3gumi•385
@takameg
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Project tracking intro component
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Any feedback/advice to improve my code is appreciated. The code became a bit unorganized. Was using position: relative; at first, but margin negative didn't work to remove the gap somehow and couldn't find other ways as well. So just changed the code using position: absolute.

Community feedback

  • Roman Filenko•3,335
    @rfilenkoPosted over 4 years ago

    Hello Meg, it's better to use flexbox or css grid for layouts in general. It easier to maintain and make web responsive with mediaqueries. Try to fix overflow issues. You can use * {outline: 1px solid tomato} to debug problem and see, which element has bigger width. Also for images set max-width: 100%.

    Roman

    1
    M3gumi•385
    @takamegPosted over 4 years ago

    @rfilenko Hi Roman, sorry for very late reply. Thank you very much for your comment. I will check.

    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