Merge pull request #1202 from AlexKVal/fix-eslint-warnings

Fix eslint warnings

.eslintrc

@@ -32,7 +32,7 @@
     "react/no-did-mount-set-state": 2,
     "react/no-did-update-set-state": 2,
     "react/no-multi-comp": 2,
-    "react/prop-types": [1, { "ignore": [ "children", "className" ] }],
+    "react/prop-types": [2, { "ignore": [ "children", "className", "style" ] }],
     "react/react-in-jsx-scope": 2,
     "react/self-closing-comp": 2,
     "react/wrap-multilines": 2,

src/Dropdown.js

@@ -247,6 +247,11 @@ Dropdown.propTypes = {
     CustomPropTypes.exclusiveRoles(MENU_ROLE)
   ]),
 
+  /**
+   * Whether or not component is disabled.
+   */
+  disabled: React.PropTypes.bool,
+
   /**
    * Align the menu to the right  side of the Dropdown toggle
    */
@@ -259,6 +264,11 @@ Dropdown.propTypes = {
    */
   open: React.PropTypes.bool,
 
+  /**
+   * A callback fired when the Dropdown closes.
+   */
+  onClose: React.PropTypes.func,
+
   /**
    * A callback fired when the Dropdown wishes to change visibility. Called with the requested
    * `open` value.

src/MenuItem.js

@@ -75,6 +75,7 @@ MenuItem.propTypes = {
   ]),
   header: React.PropTypes.bool,
   href: React.PropTypes.string,
+  target: React.PropTypes.string,
   title: React.PropTypes.string,
   onKeyDown: React.PropTypes.func,
   onSelect: React.PropTypes.func

src/Modal.js

@@ -305,8 +305,7 @@ const Modal = React.createClass({
       this.iosClickHack();
     }
 
-    this.setState(this._getStyles() //eslint-disable-line react/no-did-mount-set-state
-      , () => this.focusModalContent());
+    this.setState(this._getStyles(), () => this.focusModalContent());
   },
 
   onHide() {

src/NavDropdown.js

@@ -4,13 +4,12 @@ import Dropdown from './Dropdown';
 class NavDropdown extends React.Component {
 
   render() {
-    let { children, title, noCaret, bsStyle, ...props } = this.props;
+    let { children, title, noCaret, ...props } = this.props;
 
     return (
       <Dropdown {...props} componentClass='li'>
         <Dropdown.Toggle
           useAnchor
-          bsStyle={bsStyle}
           disabled={props.disabled}
           noCaret={noCaret}
         >
@@ -25,6 +24,7 @@ class NavDropdown extends React.Component {
 }
 
 NavDropdown.propTypes = {
+  noCaret: React.PropTypes.bool,
   title: React.PropTypes.node.isRequired,
   ...Dropdown.propTypes
 };

src/SplitButton.js

@@ -8,7 +8,14 @@ class SplitButton extends React.Component {
 
   render() {
     let {
-      children, title, onClick, target, href, bsStyle, ...props } = this.props;
+      children,
+      title,
+      onClick,
+      target,
+      href,
+      // bsStyle is validated by 'Button' component
+      bsStyle, // eslint-disable-line
+      ...props } = this.props;
 
      let { disabled } = props;
 

src/Tabs.js

@@ -106,7 +106,7 @@ const Tabs = React.createClass({
     let {
       id,
       className,
-      style, // eslint-disable-line react/prop-types
+      style,
       position,
       bsStyle,
       tabWidth,

src/Tooltip.js

@@ -60,8 +60,7 @@ const Tooltip = React.createClass({
     const style = {
       'left': this.props.positionLeft,
       'top': this.props.positionTop,
-      // we don't want to expose the `style` property
-      ...this.props.style // eslint-disable-line react/prop-types
+      ...this.props.style
     };
 
     const arrowStyle = {