@medic-code
Posted
Nice HTML structure, I completely forgot about the different assets for different screen sizes so when i saw the picture tags it made me think of something i hadn't done.
The CSS looks well structured, efficient use of CSS, by comparison I had so many different margins in the different media queries, so great to compare where i could've been more efficient. I felt the design itself was so inconsistent in terms of spacing, just gave up trying to be systematic.
For the footer overlay you could do a absolute positioned overlay using ::before and slighty reduced opacity and using z-index.
Some minor UI aspects
A comment is the hero images are not overflowing, i couldn't get this to work either on my solution but just if you weren't aware of that in the design!
The desktop hero images are slightly different in terms of height, so you could apply a specific margin for both hero images.
Marked as helpful
@medic-code bro, first of all, thank you very much for your comment
I really needed guidance because I didn't know where to go when I had already finished everything and didn't know which CSS lines to use anymore
Thank you very much for the compliment on my lines of code! One thing that helps me a lot is always trying to make it as meaningful as possible, I want my lines to actually express what they are styling but I often fail in this mission because I don't have the knowledge I need lol
Anyway, thanks for helping me with the footer, I was lost with that lol, I'll try to take a look at your code to see how you handled it
Thank you very much Mr Smith
