Skip to content

Conversation

@Bravepeanut
Copy link
Contributor

Fixes #31

@bsbeeks
Copy link
Contributor

bsbeeks commented Oct 7, 2015

The changes to Modal.js look good! I'm not sure why there are changes to .eslintrc. If you remove those, then this should be ready to merge.

@Bravepeanut
Copy link
Contributor Author

I removed the changes to .eslintrc

@bsbeeks
Copy link
Contributor

bsbeeks commented Oct 9, 2015

Looks great! Congrats on your first open source contribution! 🎉

bsbeeks added a commit that referenced this pull request Oct 9, 2015
Added CSS classes to allow styling
@bsbeeks bsbeeks merged commit aab2926 into mxenabled:master Oct 9, 2015
@Bravepeanut
Copy link
Contributor Author

Thanks for helping out!

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