kbuck2018 / kbuck2018.github.io Goto Github PK

There are many common ways to order CSS properties (randomly, alphabetically, by type). The method I suggest is by type, based on this article

Looks like you found someone's missing semicolon!

Maybe delete it?

It's really common to use a list (ul) in a nav, this would be a good place to do that!

See if you can make your CSS classes reusable and modular so that you only need 1 css file!

As applications and websites grow, it's common to break up the css into multiple files, usually following some kind of architecture or pattern. There's a ton out there about architecture, I recommend SMACSS.

The architecture I used the most was one based on what is outlined in this article. I would match it with a class naming convention called BEM:

• https://css-tricks.com/bem-101/
• https://csswizardry.com/2013/01/mindbemding-getting-your-head-round-bem-syntax/
• https://robots.thoughtbot.com/keeping-the-frontend-modular-with-bem

Here is a final article on writing clean/organized css: https://philipwalton.com/articles/css-architecture/

It seems you're using the h5s here for sizing/styling? Consider instead using classes and css for this. I think this block of HTML could also be better divided up. Perhaps you could put the first set of links in a list and the second set in a separate list?