Skip to content

Working on dropdown#86

Open
Julian-Alexander wants to merge 5 commits intobloominstituteoftechnology:masterfrom
Julian-Alexander:master
Open

Working on dropdown#86
Julian-Alexander wants to merge 5 commits intobloominstituteoftechnology:masterfrom
Julian-Alexander:master

Conversation

@Julian-Alexander
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@frogr frogr left a comment

Choose a reason for hiding this comment

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

Hey Julian!

This is awesome. You did a very great job!

I don't have any real criticisms here. I think you knocked it out of the park.

The code is clean, your markup is solid, good naming conventions, and it all matches the spec. I have no complaints!

You knocked this one out of the park. Nice work!

@@ -0,0 +1,13 @@
class dropDown {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Loved that you used class syntax!

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