@visualdenniss
Posted
Hey there,
nice work!
-
I'd suggest wrapping the content into <main> tag as each page / body should have one <main> tag. Also having a container would make your control over the children easier, as it is now, both image and section are direct children of the body, making it harder to control.
-
I'd suggest using max-width: 890px instead of fixed width for the image, so it was be more responsive in general.
Hope you find this feedback helpful!
Marked as helpful