Relaxing emptyLabel's PropType requirements

[clayjonesthomas]

It appears that giving emptyLabel a React component renders that component just fine. I didn't look too far into the reasoning behind why emptyLabel is required to be a string, but I believe React components should be allowed as well. Is there another way to give emptyLabel React components that I am missing, or a reason why React components are not accepted?

For convenience, the line with the PropType.

react-bootstrap-typeahead/src/containers/typeaheadContainer.js

Line 410 in 60ea7ce

emptyLabel: PropTypes.string,

[ericgio]

No good reason other than a string seemed like the most likely use case and I hadn't encountered a need for more. I don't see why a component shouldn't be allowed.

[ericgio]

For clarification, passing a component to emptyLabel now works just fine; the only issue is a React warning. This is only in developer warning, however, and shouldn't show up in production or have any other effects on how the component behaves.

[ericgio]

Should probably just be

emptyLabel: PropTypes.node,

[ericgio]

This should be resolved as of v2.3.0