100vh is too tall for the browser

On the example page, when they use 100vh, the background fits the browser window. Where I’ve used it the background spills over. Can anyone see where I’m going wrong?

I apologise if my code looks rather messy, I am fairly new to HTML & CSS.

Any other constructive criticism is also welcome.

Thank you.

Spent hours trying to find this and then discover the issue right after asking the question.

So, it looks like by using padding to vertically align my text, I was forcing the height of the background to be larger.