-
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
Brenna's Static Site #19
base: master
Are you sure you want to change the base?
Conversation
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.
Did you end up using all the html files? I couldn't get to the bio, blog, or portfolio pages from the index page. Nice job making your css dry!
I'm Brenna (not Brenda or Brianna or Greta) Darroch (pronounced like Arrow)! I love making art, reading, writing, and coding. I'm currently a student at Ada Developers Academy, learning Ruby, HTML/CSS, Ruby on Rails, and Javascript. On the side, I'm teaching myself game programming. | ||
</p> | ||
<p> | ||
Fun fact: I know the following musicals by heart: Grease, Sound of Music, My Fair Lady, Phantom of the Opera, West Side Story, and Fiddler on the Roof. Currently working on Hamilton. :) |
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.
👍 You're my hero!!! I absolutely LOVE musicals!
<img src="images/portfolio.png" alt="Art Portfolio" /> | ||
<p class="desc"> | ||
Using HTML, CSS, and Javascript Lightbox, I coded my art portfolio site. | ||
</p> |
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.
This is repetitive, but I don't see it on your actual webpage. Not sure if I'm looking at the right place 🤔
<link href="styles/normalize.css" rel="stylesheet"> | ||
<link href="styles/styles.css" rel="stylesheet"> | ||
|
||
<!-- favicon didn't work! |
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.
favicons are such a pain, amiright??
color: #09B3C4; | ||
} | ||
|
||
h1 a, h2, h4 { |
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.
nice job staying DRY!
} | ||
|
||
*, *:before, *:after { | ||
box-sizing: inherit; |
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'm not sure that this line is necessary...does this do the same as lines 8-10?
padding: 0 2.5%; | ||
} | ||
|
||
.prof { |
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.
does this refer to any of the classes in your html file? Can't find its reference anywhere
} | ||
|
||
section.contact img { | ||
width: 50px; |
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.
maybe even add the footer img selector here to make it more dry!
@@ -0,0 +1,139 @@ | |||
/* color1: #09B3C4 */ |
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.
You might not use it at ADA but if you use sass (different but similar to css) you can make these constants so you can declare them once and refer to them everywhere! (makes changing the color soooo easy in the future!)
Static Site
Congratulations! You're submitting your assignment!
Comprehension Questions