fix use special navItem for signin button

common/app/components/Nav/Nav.jsx

@@ -1,5 +1,6 @@
 import React from 'react';
 import { Nav, Navbar, NavItem } from 'react-bootstrap';
+import FCCNavItem from './NavItem.jsx';
 
 export default class extends React.Component {
   constructor(props) {
@@ -33,12 +34,12 @@ export default class extends React.Component {
       );
     } else {
       return (
-        <NavItem
+        <FCCNavItem
           className='btn signup-btn signup-btn-nav'
           eventKey={ 2 }
           href='/login'>
           Sign In
-        </NavItem>
+        </FCCNavItem>
       );
     }
   }

common/app/components/Nav/NavItem.jsx

@@ -1,65 +1,86 @@
 import React from 'react';
-import classnames from 'classnames';
+import classNames from 'classnames';
+import BootstrapMixin from 'react-bootstrap/lib/BootstrapMixin';
 
-export default class extends React.Component {
+export default React.createClass({
-  constructor(props) {
+  displayName: 'FCCNavItem',
-    super(props);
+  mixins: [BootstrapMixin],
-  }
 
-  static displayName = 'NavItem'
+  propTypes: {
-  static propTypes = {
-    onSelect: React.PropTypes.func,
     active: React.PropTypes.bool,
+    'aria-controls': React.PropTypes.string,
     disabled: React.PropTypes.bool,
-    href: React.PropTypes.string,
-    title: React.PropTypes.string,
     eventKey: React.PropTypes.any,
-    target: React.PropTypes.string
+    href: React.PropTypes.string,
-  }
+    linkId: React.PropTypes.string,
+    onSelect: React.PropTypes.func,
+    role: React.PropTypes.string,
+    target: React.PropTypes.string,
+    title: React.PropTypes.node
+  },
+
+  getDefaultProps() {
+    return {
+      href: '#'
+    };
+  },
 
   handleClick(e) {
     if (this.props.onSelect) {
       e.preventDefault();
 
       if (!this.props.disabled) {
-        this.props.onSelect(
+        this.props.onSelect(this.props.eventKey, this.props.href, this.props.target);
-          this.props.eventKey,
-          this.props.href,
-          this.props.target
-        );
-      }
       }
     }
+  },
 
   render() {
-    const {
+    let {
+      role,
+      linkId,
       disabled,
       active,
       href,
       title,
       target,
       children,
+      'aria-controls': ariaControls,  // eslint-disable-line react/prop-types
+      className,
+      ...props
     } = this.props;
 
-    const classes = {
+    let classes = {
-      'active': active,
+      active,
-      'disabled': disabled
+      disabled
     };
 
+    let linkProps = {
+      role,
+      href,
+      title,
+      target,
+      id: linkId,
+      onClick: this.handleClick,
+      ref: 'anchor'
+    };
+
+    if (!role && href === '#') {
+      linkProps.role = 'button';
+    }
+
     return (
       <li
-        className={ joinClasses(props.className, classSet(classes)) }
+        {...props}
-        { ...this.props }>
+        role='presentation'>
         <a
-          href={href}
+          { ...linkProps }
-          title={title}
+          aria-selected={ active }
-          target={target}
+          aria-controls={ ariaControls }
-          className={ this.props.aClassName }
+          className={ className }>
-          onClick={this.handleClick}
-          ref="anchor">
           { children }
         </a>
       </li>
     );
   }
-}
+});