Skip to content

Add Modal portal className #767

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

Merged

Conversation

interactivellama
Copy link
Contributor

@interactivellama interactivellama commented Nov 10, 2016

For @mvantassel

Fixes: #765 and closed #671

@interactivellama interactivellama modified the milestones: 2016.10b-design-system-react, 2016.11b-design-system-react Nov 11, 2016
@interactivellama
Copy link
Contributor Author

@mvantassel Do you feel comfortable reviewing this? My other MC-UX person is now working on another project.

@interactivellama interactivellama merged commit 31088c4 into salesforce:master Nov 15, 2016
@mvantassel
Copy link

My apologies @interactivellama. I've been focused on Guardrails. This PR looks good. Thanks for completing this work. When we get back to working on our original project we'll be able to remove our workaround!

@interactivellama interactivellama deleted the modal-portal-class branch December 14, 2016 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add portalClassName prop for high level wrapper Add containerClassName to "modal" menu-dropdown and actual modal
3 participants