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

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.

<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

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!

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

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.

padding: 2%
}

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

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.

}
blockquote:after {
content: close-quote;
}
Copy link

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.

justify-content: space-evenly;
}

.footer-sect {
Copy link

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

@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