[fixed] Fix propTypes for overlays

Fixes react-bootstrap#173

Author: Jimmy Jia

src/Popover.js

@@ -9,8 +9,12 @@ const Popover = React.createClass({
     placement: React.PropTypes.oneOf(['top', 'right', 'bottom', 'left']),
     positionLeft: React.PropTypes.number,
     positionTop: React.PropTypes.number,
-    arrowOffsetLeft: React.PropTypes.number,
-    arrowOffsetTop: React.PropTypes.number,
+    arrowOffsetLeft: React.PropTypes.oneOfType([
+      React.PropTypes.number, React.PropTypes.string
+    ]),
+    arrowOffsetTop: React.PropTypes.oneOfType([
+      React.PropTypes.number, React.PropTypes.string
+    ]),
     title: React.PropTypes.node
   },
 
@@ -21,19 +25,19 @@ const Popover = React.createClass({
   },
 
   render() {
-    let classes = {
+    const classes = {
       'popover': true,
       [this.props.placement]: true,
       'in': this.props.positionLeft != null || this.props.positionTop != null
     };
 
-    let style = {
+    const style = {
       'left': this.props.positionLeft,
       'top': this.props.positionTop,
       'display': 'block'
     };
 
-    let arrowStyle = {
+    const arrowStyle = {
       'left': this.props.arrowOffsetLeft,
       'top': this.props.arrowOffsetTop
     };

src/Tooltip.js

@@ -9,8 +9,12 @@ const Tooltip = React.createClass({
     placement: React.PropTypes.oneOf(['top', 'right', 'bottom', 'left']),
     positionLeft: React.PropTypes.number,
     positionTop: React.PropTypes.number,
-    arrowOffsetLeft: React.PropTypes.number,
-    arrowOffsetTop: React.PropTypes.number
+    arrowOffsetLeft: React.PropTypes.oneOfType([
+      React.PropTypes.number, React.PropTypes.string
+    ]),
+    arrowOffsetTop: React.PropTypes.oneOfType([
+      React.PropTypes.number, React.PropTypes.string
+    ])
   },
 
   getDefaultProps() {
@@ -20,18 +24,18 @@ const Tooltip = React.createClass({
   },
 
   render() {
-    let classes = {
+    const classes = {
       'tooltip': true,
       [this.props.placement]: true,
       'in': this.props.positionLeft != null || this.props.positionTop != null
     };
 
-    let style = {
+    const style = {
       'left': this.props.positionLeft,
       'top': this.props.positionTop
     };
 
-    let arrowStyle = {
+    const arrowStyle = {
       'left': this.props.arrowOffsetLeft,
       'top': this.props.arrowOffsetTop
     };

test/OverlayTriggerSpec.js

@@ -1,6 +1,8 @@
 import React from 'react';
 import ReactTestUtils from 'react/lib/ReactTestUtils';
 import OverlayTrigger from '../src/OverlayTrigger';
+import Popover from '../src/Popover';
+import Tooltip from '../src/Tooltip';
 
 describe('OverlayTrigger', function() {
   it('Should create OverlayTrigger element', function() {
@@ -161,4 +163,34 @@ describe('OverlayTrigger', function() {
       });
     });
   });
+
+  describe('overlay types', function() {
+    [
+      {
+        name: 'Popover',
+        overlay: (<Popover>test</Popover>)
+      },
+      {
+        name: 'Tooltip',
+        overlay: (<Tooltip>test</Tooltip>)
+      }
+    ].forEach(function(testCase) {
+      describe(testCase.name, function() {
+        let instance, overlayTrigger;
+
+        beforeEach(function() {
+          instance = ReactTestUtils.renderIntoDocument(
+            <OverlayTrigger trigger="click" overlay={testCase.overlay}>
+              <button>button</button>
+            </OverlayTrigger>
+          );
+          overlayTrigger = React.findDOMNode(instance);
+        });
+
+        it('Should handle trigger without warnings', function() {
+          ReactTestUtils.Simulate.click(overlayTrigger);
+        });
+      });
+    });
+  });
 });