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

Option to toggle portal when clicking openByClickOn element #104

Closed
wants to merge 9 commits into from

Conversation

rdiazv
Copy link

@rdiazv rdiazv commented Oct 17, 2016

Adds togglesOnClick prop, allowing the openByClickOn element to toggle the portal on subsequent clicks instead of just keep it open.

@tajo
Copy link
Owner

tajo commented Dec 29, 2016

Looking good! Can you please resolve conflicts and fix tests?

Sorry for not responding sooner.
Thank you!

@tajo tajo added this to the v4 milestone Dec 29, 2016
@rdiazv
Copy link
Author

rdiazv commented Jan 20, 2017

Done.

@tajo
Copy link
Owner

tajo commented Oct 1, 2017

Hey, please check the new major version (complete rewrite) of react-portal: #157

It's React v16 only since its uses the new official Portal API. There is the first beta released and I would like to get your feedback. I don't have bandwidth to maintain v3 which is very different and full of hacks.

If you feel your PR still applies, please rebase it against the master and re-open it. Sorry for the lack of response in past! Thanks!

@tajo tajo closed this Oct 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants