docs(Popup): make allowed content types statically analyzable

[dvdzkwsk]

This is presumably an issue with all propType definitions that reuse propTypes defined in another variable. I traced it down to react-docgen being unable to parse the union type because it viewed it as dynamic, which is technically true because that type definition is referenced from a variable and as such the propType is resolved at runtime.

Before

After

[dvdzkwsk]

Also this looked to be in error, since it is not an enum but a propType.

[codecov-io]

Current coverage is 99.73% (diff: 100%)

Merging #990 into master will not change coverage

@@             master       #990   diff @@
==========================================
  Files           140        140          
  Lines          2304       2304          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           2298       2298          
  Misses            6          6          
  Partials          0          0          

Powered by Codecov. Last update fa9175a...b35fa51

[levithomason]

Yikes, thanks. Since node is "anything that can be rendered" it includes string, so we can actually simplify this to PropTypes.node. React PropTypes docs:

// Anything that can be rendered: numbers, strings, elements or an array
// (or fragment) containing these types.
optionalNode: React.PropTypes.node,

[dvdzkwsk]

Good call, was not even paying attention to the types to be honest. Patch incoming.

[dvdzkwsk]

Pushed and renamed commit accordingly

[levithomason]

Thanks will merge on pass.

[levithomason]

Released in semantic-ui-react@0.62.0.