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

All comments

  • Adrian Maj 170

    @AdrianMaj

    Posted

    I think everything looks good there, it is accessible and responsive. You are using semantic HTML, layout looks good on phones. I have no reservations for that. Thanks!

    0
  • Adrian Maj 170

    @AdrianMaj

    Posted

    I think your heading text color should be hsl(217, 19%, 35%) on the white background cards, also the photo size and font-weights are not similar to original preview. And you are missing background image on purple card. You can also get some work with paddings and margins.

    And for your question , I created empty div then in SCSS use those: background-image, background-size, background-repeat: no-repeat; background-position, this should work better than using position relative and absolute

    Marked as helpful

    0