Skip to content

Commit 3c85424

Browse files
committed
Fix evaluated Wrapper type
React.createElement expects string or React.Component in first argument. React.DOM.div and React.DOM.span are neither strings nor Components. They are wrappers around React.createElement. Also, they are deprecated. Read more: https://facebook.github.io/react/docs/react-api.html#createelement For our case, we could safely pass strings to React.createElement, so, no need to use React.DOM.div etc
1 parent c37e063 commit 3c85424

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

src/index.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,7 @@ class ReactTooltip extends Component {
5858
wrapper: 'div'
5959
};
6060

61-
static supportedWrappers = {
62-
'div': React.DOM.div,
63-
'span': React.DOM.span
64-
};
61+
static supportedWrappers = ['div', 'span'];
6562

6663
constructor (props) {
6764
super(props)
@@ -433,8 +430,10 @@ class ReactTooltip extends Component {
433430
{'type-light': this.state.type === 'light'}
434431
)
435432

436-
let Wrapper = ReactTooltip.supportedWrappers[this.props.wrapper]
437-
if (!Wrapper) Wrapper = ReactTooltip.supportedWrappers['div']
433+
let Wrapper = this.props.wrapper
434+
if (ReactTooltip.supportedWrappers.indexOf(Wrapper) < 0) {
435+
Wrapper = 'div'
436+
}
438437

439438
if (html) {
440439
return (

0 commit comments

Comments
 (0)