Only call onClose when Portal is actually closed

[Andreyco]

Fixes #95

[tajo]

I want to merge this. Could you please add an test checking this behaviour and resolve conflicts? Thanks!

[mzygmunt]

Ohh is there any chance to merge this patch?

[Andreyco]

Sorry, somehow I missed the comment where about express will to merge...
Will add tests and resolve conflicts asap.

[mzygmunt]

Thanks for quick response.

[likerRr]

@Andreyco any updates on tests?

[tajo]

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!