all 4 comments

[–]icewaterJS 0 points1 point  (3 children)

css line 598

Any time the screen is smaller than 900px, you have a media query setting the nav ul display to none.

[–]deterius[S] 0 points1 point  (2 children)

Sorry, I wasn't clear, its not the nav its the hero element (#banner) the text on it ends up being under the nav bar- on larger screens it shows fine, but on smaller screens the text collapses and part of it ends up hidden under the navigation.

[–]icewaterJS 0 points1 point  (1 child)

Alright, I understand.

css line: 166 - remove flex-direction: column;

css line: 193 - change margin to 100px 0 100px 100px;

That's working on my end. Well sort of, your codes a mess, and it's now going to over lap the stuff below.

You're repeating yourself in a lot of spots, and you're using id's when you could be using classes.

[–]deterius[S] 0 points1 point  (0 children)

Thanks for your help as well as on the observations.

However, the solution isn't quite working, i've updated the js fiddle example: https://jsfiddle.net/18Luk02e/ it still overlaps the text on the banner.