Skip to content

Devon Bacon - Components - BEM#62

Open
devbacon wants to merge 18 commits into
bloominstituteoftechnology:masterfrom
devbacon:master
Open

Devon Bacon - Components - BEM#62
devbacon wants to merge 18 commits into
bloominstituteoftechnology:masterfrom
devbacon:master

Conversation

@devbacon

Copy link
Copy Markdown

No description provided.

@BonnW BonnW left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey Devon!

Great work on the project. You've definitely got a strong understanding of the content. I especially love the all of the reps you're getting since you're helping others as well as going above and beyond on your own.

The only comment I have for you is one extremely minor styling issue. In your styles.css your body tag is indented further than everything else.

It's important to try your best and stay consistent throughout the entirety of your code.

However, again, this is a VERY small issue. Otherwise, everything is perfect. Fantastic job!

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