Skip to content

Branches - Emily Ball#28

Open
eaball35 wants to merge 2 commits intoAda-C12:masterfrom
eaball35:master
Open

Branches - Emily Ball#28
eaball35 wants to merge 2 commits intoAda-C12:masterfrom
eaball35:master

Conversation

@eaball35
Copy link
Copy Markdown

Startrly

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
Describe a time you chose to use flexbox I used flexbook to space the sponsor images evenly and to have them wrap with screen size.
Describe a time you chose to use CSS Grid I used grid when I spaced the nav text so that I could space the last two at the end and the first at the beginning.
What was a challenge you overcame in this project? Alignment is hard and knowing when is best to use grid vs flex. I generally became more comfortable using these concepts throughout the project.
What concept did you get the most clarity on through Startrly? I feel way more comfortable with flex, I think I could use additional practice with grid.

Comment thread index.html
<meta charset="utf-8">
<LINK REL=StyleSheet HREF="normalize.css">
<LINK REL=StyleSheet HREF="styles.css">
<link href="https://fonts.googleapis.com/css?family=Mansalva&display=swap" rel="stylesheet">
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great job using the fonts!

Comment thread index.html
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
</blockquote>
<p>
<p class="quote-author">
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i really like how youre using your classes.

Comment thread styles.css
padding: 2%
}

section:nth-last-of-type(even) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alright! great use of this element.

Comment thread styles.css
}
blockquote:after {
content: close-quote;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is so good. i'm going to take this into consideration next time i work with quotes.

Comment thread styles.css
justify-content: space-evenly;
}

.footer-sect {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you gave your footer alot of attention and it works beautifully.

Copy link
Copy Markdown

@OhCloud OhCloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall great work! Your layout is spot on and everything looks great. I did notice you didn't use ids and I liked that you didn't have to, you kept everything in classes and re-used it well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants