React-modal: Please remove unused dependency react-dom-factories

[bjbrewster]

Summary:

react-dom-factories was used in lib/components/Modal.js but removed in conversion to ES2015 src/components/Modal.js.

Steps to reproduce:

1. Search repo for react-dom-factories
2. See it is not used by any source files.

Expected behavior:

Should not be installed.

Additional notes:

I realise this dependency is not imported and so is not in dist build or my app's bundle. I wondered why it was in my apps node_modules and saw the repos readme says it should not be used and wondered why react-modal was depending on it to find out it no longer uses it.

[diasbruno]

Hi @bjbrewster, just give a thumbs up and I'll merge (I couldn't assign you on the PR).

[bjbrewster]

Thanks!

[tino-junge]

Thanks for fixing this @diasbruno ! Any ideas when the next npm release for react-modal is planned?

[diasbruno]

Released v2.3.3. Sorry for taking so long.