@hardy333
Posted
No criticism whatsoever, You killed it.
One small thing - on small screen size .container element becomes smaller than screen size and background images does not take whole space on screen, so maybe moving those background images to body element might be better I think .
Marked as helpful
@nogyuuu
Posted
@hardy333 thanks so much, i will do that right away! do you think it would be best to put it directly into the body element or to create a new div wrapping the whole thing?
@hardy333
Posted
@nogyuuu Both ways will be fine. But creating wrapper container as you have already done will be a little bit more robust and modular I guess, in case if you want to copy and paste html and css in other pages.
Sometimes I also using section element in main element and this section element takes all the viewport width space and that case I will use background properties in section element.
Marked as helpful
