Skip to content

Branches - Erika#46

Open
emaust wants to merge 3 commits intoAda-C12:masterfrom
emaust:master
Open

Branches - Erika#46
emaust wants to merge 3 commits intoAda-C12:masterfrom
emaust:master

Conversation

@emaust
Copy link

@emaust emaust commented Sep 20, 2019

Startrly

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
Describe a time you chose to use flexbox I used it for the bears
Describe a time you chose to use CSS Grid I used grid for the nav bar and the why section
What was a challenge you overcame in this project? Forgetting to commit to git -_-
What concept did you get the most clarity on through Startrly? Remembering how to use grid and when it is appropriate to use it instead of flexbox

<nav>
<ul>
<li>
<li class="b">

Choose a reason for hiding this comment

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

I like how you label classes with letters - this makes it easy to follow vs descriptive class names, which could mean one thing to you vs the reader.

color: white;
padding: 1em;
display: grid;
grid-template-columns: repeat(6 1fr);

Choose a reason for hiding this comment

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

ooh I haven't seen this
'repeat(6 1fr);

}

.quote {
font-family: 'Lato', cursive;

Choose a reason for hiding this comment

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

Nice usage of multiple fonts!

<li>
<img src="https://placebear.com/150/150" alt="profile for dan dragon" />
<img class="bear-img" src="https://placebear.com/150/150" alt="profile for dan dragon" />
<span>Dan Dragon</span>

Choose a reason for hiding this comment

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

Nice usage of span

background-color: #354d60;
color: white;
padding-bottom: 1
} No newline at end of file

Choose a reason for hiding this comment

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

add 1 new line at the end of each file you create ( I think this is a design thing)

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