search

b-ivey / css-project

0 stars 0 forks source link

Jessica Sanz #5

Open Jess-Sanz opened 4 years ago

Jess-Sanz commented 4 years ago

https://github.com/Jess-Sanz/CSS-Project

b-ivey commented 4 years ago

YOU DID IT!!! Considering you didn't have much a week ago, I cannot express how proud I am of you for getting this project done.

That said, some issues you should look out for:

  1. You are missing the max-width of 960 px for the text.

  2. Your name should be much bigger. You can control the size of the H1 tag to be a specific size.

  3. I think the photo should be above all the text. So the contact info can go beneath the photo, and consider adding a color background behind it so it pops out.

  4. The gray background is a tad dark. Try a light color -- like a light blue or another complimentary color to play off your dress.

When you create your website, you will really need to organize your content and add more images. You can include an iframe tag to embed videos too.