-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Alena's Static-Site #25
base: master
Are you sure you want to change the base?
Conversation
…ine for main body content
…ner, added seattle background and banner
…ain tag, header and footer, added lorem ipsum
…d text floating around those images
… hand drawn path now! :D), added styling for paper airplane images
…f title, changed size of git icon
…rs, and aded classes and ids
Static SiteWhat We're Looking For
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great Job! The site looks amazing and speaks to your personality and design style. Feel free to reach out to me if you have any questions :)
|
||
<body> | ||
<header class="header-and-footer"> | ||
<a id="home" href="index.html"> Home </a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's more semantic to not include spaces on either end of the text in between the html tags ... Ex: Home
</head> | ||
|
||
<body> | ||
<header class="header-and-footer"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like how you used the same class name for the header and footer. Great Job keeping your css dry.
</nav> | ||
</header> | ||
|
||
<main> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like your design decision to break up where you came from, where you are and where your going 👍
</header> | ||
|
||
<main> | ||
<section class="background" id="prague"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since the article and section here are both part of the same where I came from part of the page. How do you think you could show that semantically using html?
|
||
<main> | ||
<section class="background" id="books"> | ||
<h1 class="banner" id="books-banner"> Book Adventures </h1> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like your background image and font selection. Do you think the contrast between them should be increased for user experience or accessibility reasons?
width: 100%; | ||
top: 0; | ||
z-index: 100; | ||
display: flex; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great use of flex box!
color: #ffeecc; | ||
} | ||
|
||
a:hover { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
always like hover overs on links so the user knows it's a link. good job.
} | ||
|
||
#content { | ||
animation: fadein 4s; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this, it's subtle but impactful.
|
||
#future { | ||
background: url("../images/silicon-valley.jpg"); | ||
background-size: cover; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lines 29-32 are similiar to what's under #seattle.... Is there a way that you can think of to not duplicate code and dry this up?
padding-bottom: 1%; | ||
} | ||
|
||
article:nth-of-type(odd) .image-box { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great use of :nth-of-type
Static Site
Congratulations! You're submitting your assignment!
Comprehension Questions