Provider only accepts a single child

[TimoStaudinger]

Currently, the Provider component accepts only a single child:

render() {
  return React.Children.only(this.props.children);
}

This somewhat messes with our implementation, and also seems to render the example in the readme invalid:

import Toolbar from './Toolbar';
import Feature from './Feature';

import { Provider } from 'react-slot-fill';

const App = () =>
  <Provider>
    <Toolbar />
    <Feature />
  </Provider>

ReactDOMFiber.render(
  <App />,
  document.getElementById('root')
);

---

Did I misunderstand any of this? Is there a reason why the library would not allow multiple children for Provider?

[camwest]

The Provider currently doesn’t render any html elements so it conforms to the React 15.x API where it creates a context and then renders the provided child inside that context.

The docs are out of date.

We can update to React 16.x only and change this so that Provider can handle multiple children. Want to make the change?

[TimoStaudinger]

Yep, I'll try to find some time this week to submit a PR for this.

From our perspective, depending on React 16 is fine, as we are already on 16 anyhow.

[camwest]

Ok awesome!