Kenneth austin flex - #1
Conversation
LeTanque
left a comment
LeTanque
left a comment
There was a problem hiding this comment.
Links to images are broken (wrong src path).
Nav link doesn't work.
Nav formatting doesn't match design file.
About page formatting doesn't match design file.
Doesn't meet MVP.
| 1. If you were to describe semantic HTML to the next cohort of students, what would you say? | ||
|
|
||
| Makes co | ||
|
|
| <div> | ||
| <div class="top-img"> | ||
| <img class="about-header-img" src="/img/jumbo-about.png" > | ||
| </div> |
There was a problem hiding this comment.
Anchor tag src path is incorrect.
/img/...
doesn't work and is not the same as
./img/...
Which is what you want.
/img refers to the root directory. ./img refers to the current directory.
| <a href="#">Contact</a> | ||
| </footer> | ||
| </body> | ||
|
|
There was a problem hiding this comment.
The bulk of this page should be wrapped in a container. Like a div or a main tag, or something.
That container should be 100% wide, max 800px, and centered.
| <button>Learn More</button> | ||
| </div> | ||
| <div id="Collab"> | ||
| <img src="img/about-meeting.png" alt="strategy"> |
There was a problem hiding this comment.
these images should be 100% wide so that they take up their parent container width, only.
2 participants
No description provided.