connor-cahill / connor-website Goto Github PK

If you set the font on the body all elements would use the chosen font. You can then set the elements that need a different font to something else. Setting the most common font on your page in the body ensures that everything uses the font, is less work, and easier to edit.

Run your page through the validator. You did pretty good only one error. See if you can find it and fix it.

This byline isn't a heading. A heading is something begins a section, it's the title. This should just be a div or a paragraph. If the font is too small style it.

I feel this might look better not bold. With the two lines in bold it's a little loud.

You can just set this as margin: 0; Anytime you have 0 as a value you don't need a unit! 0 is the same for all units. Rather than setting this on three or four lines set all four at once as shown.