Conversation
Static SiteWhat We're Looking For
|
| </nav> | ||
| <div class="white-space-bottom"></div> | ||
| </header> | ||
| <body> |
There was a problem hiding this comment.
incorrect use of the <body> tag!
Very confusing, but <body> should describe where most of the HTML lives. This is in contrast to our <head> tag, which describes meta data for the website.
You'd probably want to use <main> here instead
| <link href="../stylesheets/styles.css" rel="stylesheet"/> | ||
| <link href="https://fonts.googleapis.com/css?family=Maven+Pro" rel="stylesheet"/> | ||
| </head> | ||
| <header> |
There was a problem hiding this comment.
I'd expect a <body> tag right above this line, after closing head (</head>)
| </li> | ||
| </ul> | ||
| </nav> | ||
| <div class="white-space-bottom"></div> |
There was a problem hiding this comment.
instead of using <div class="white-space-bottom"></div> it may be better to give a bottom margin to your nav
|
|
||
| a #scrabble:hover { | ||
| opacity: 0.1; | ||
| border: solid; |
There was a problem hiding this comment.
adding a border on :hover actually shifts the content box of this a little bit, so the images shift on hover
| September 17, 2017 | ||
| </p> | ||
| <p class="blog-entry"> | ||
| It's probably because you tried to give a property a value with an equals operator instead of a colon. Once you fix that, all your dreams will come true. |
|
Nice website. Overall, I have concerns that you didn't get to appropriately place your |
2 participants
Static Site
Congratulations! You're submitting your assignment!
Comprehension Questions