Your CSS could use some formatting. There are selectors that are immediately followed by the brace:
There are also places where the opening brace are below the selector:
background-color: rgb(217,236,242) ;
You should always aim to make your code consistent. In general, you should write CSS like this:
Notice: There’s one blank line between CSS rules. The properties are indented (how much to indent is up to you, but it should be consistent). The selector is preceded by a space, then the opening brace. The closing brace is on its own line. There’s a space between a property and its value. Sometimes spaces are just easy to miss though (I do sometimes )
Same with the HTML. The code is not properly indented. Other than one unclosed
<h5> element around line 46, it doesn’t look like you have missing/misplaced tags though, but it’s easier to avoid/spot them if you indent your code, and it makes reading your code much easier.
Instead of using two
<h5> elements for “Aspiring Web Developer Based in Seattle”", you can insert a
<br> between “Aspiring” and “Web”.
alt attribute is required for
<li> elements can be direct children of
<ul> elements (near the bottom of your code). Instead of
<a href="#"><li>...</li></a>, you should use
There’s also a Font Awesome icon for freeCodeCamp (
<i class="fa fa-free-code-camp"></i>) .
The contact link doesn’t work. You seem to have forgotten to give that section its
I can’t click on the nav links on small screens? It looks like the navbar is obscured by the dark overlay.