C17 (Sea Turtles) - Shannon Bellemore #107
Conversation
| </head> | ||
| <body> | ||
|
|
||
| <header> |
There was a problem hiding this comment.
Page is clean, semantic and readable. Good job!
| <section class="home"> | ||
| <h2>Featured Project</h2> | ||
| <a href="https://github.com/eggoweggo/task-list-api"> | ||
| <img src="/images/task-list.png" alt="Cartoon cat sweeping with broom"> |
There was a problem hiding this comment.
great use of the alt attribute with descriptive text
| <main> | ||
| <section class="portfolio"> | ||
| <h2>Personal Portfolio Project</h2> | ||
| <a href="https://github.com/eggoweggo/personal-portfolio-site"> |
There was a problem hiding this comment.
you could add the target attribute to your links and it will open them in a new tab. This keeps the user on your page and be able to access other content
an example is here https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_a_target
| color: #3e3e3e; | ||
| font-weight: 300; | ||
| background-color: #A3AC9C | ||
| /* background: url('../images/background.png') */ |
| </nav> | ||
| </header> | ||
| <main> | ||
| <p>My name is Shannon Bellemore and I am currently a software developer student at Ada Developers Academy. |
There was a problem hiding this comment.
I would add some padding around this text. It was hard to read the beginning of it on safari.
| list-style-type: none; | ||
| } | ||
|
|
||
| a { |
There was a problem hiding this comment.
to make it more user friendly I would add an effect to your links so folks know its a link. Maybe when you hover the mouse could change to a pointer or the text could change colors.
Personal Portfolio Site
Congratulations! You're submitting your assignment!
Comprehension Questions