Well, I got the test to pass, but it messed up your layout a little bit, so you’ll have to play with it.
Basically I noticed the test values were only off by 6px. Yours was bigger than it was supposed to be.
2. The height of the welcome section should be equal to the height of the viewport.
The height of #welcome-section is not equal to the height of the viewport : expected 470 to be close to 464 +/- 0
So I started removing margin and paddings, but found that what fixed the test was removing the borders from the wecome-section class.
After removing those two lines the test passed, so based on your box-sizing I’m guessing it adds to the size of the section. Now after removing that the opaque black no longer goes to the top, so you’ll have to play with that… but as I need to step away at least wanted to share what I found.
Oh, since I mentioned box-sizing… I also was able to fix it by changing welcom section to border-box by adding this line to it.
If you’ve never heard of that, look it up… it determines whether padding and borders adds to the size of a section, or if it gets placed inside of the section… I always prefer border-box because other methods don’t seem logical to me. By adding that to your welcome-section, the borders don’t increase the size, so you can keep your border statements.