Feedback is very welcome!
Adrian Maj
@AdrianMajAll comments
- @eldmarSubmitted 12 months ago@AdrianMajPosted 12 months ago
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 - @id-dev3Submitted over 1 year ago
Hey guyz! Can you show me how to put the quotation mark in the first purple grid? I try to use the
postion: relative;
andpostion: absolute;
but it really sucks Thanks for your help!@AdrianMajPosted over 1 year agoI 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 helpful0