Skip to content
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

Sockets - Kate N #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

KateAnnNichols
Copy link

Startrly

Congratulations! You're submitting your assignment.

Comprehension Questions

Feature Feedback
How are CSS Grid & Flexbox similar
Give one example where you choose Flexbox over Grid
Where did you choose to use CSS Grid and why did you make that decision
What was the hardest part of this assignment, and why was it difficult
What concept did you get the most clarity on through Startrly?

Copy link

@alexandria7 alexandria7 left a comment

Choose a reason for hiding this comment

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

Kate great work!!! I love your overall organization using CSS grid and think your code is pretty DRY :)

width: 100%;
height: 90%;
display: grid;
grid-template: auto auto / auto auto auto auto auto auto;

Choose a reason for hiding this comment

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

Love the use of auto!

</blockquote>
<p>
Mx. Ipsum
-- Mx. Ipsum

Choose a reason for hiding this comment

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

I think there is a way to add a "--" in css instead of changing the html directly! I would try using the pseudo-class ":before" with "content: '--'"


section.sponsors ul {
display: grid;
grid-template: auto auto / auto auto auto auto;

Choose a reason for hiding this comment

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

I really love your use of grid in this section! It organizes all of the list items in the configuration you want and doesn't allow for things to move out of place!

height: 40%;
display: grid;
grid-template: auto auto 40% / auto;
justify-items: center;

Choose a reason for hiding this comment

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

Does "justify-items" work without "display: flex"?

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