From f25d850c927dbaaa0ed515bf7609e79ab24b6834 Mon Sep 17 00:00:00 2001
From: Berkeley Martinez <jn.martinez110@gmail.com>
Date: Sun, 8 Jan 2017 19:58:45 -0800
Subject: [PATCH] fix(nav): add noop for onToggle to remove warning

---
 common/app/components/Nav/Nav.jsx | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/common/app/components/Nav/Nav.jsx b/common/app/components/Nav/Nav.jsx
index 2894f1762d..03a97a9462 100644
--- a/common/app/components/Nav/Nav.jsx
+++ b/common/app/components/Nav/Nav.jsx
@@ -10,6 +10,7 @@ import {
   Navbar,
   NavbarBrand
 } from 'react-bootstrap';
+import noop from 'lodash/noop';
 
 import navLinks from './links.json';
 import AvatarPointsNavItem from './Avatar-Points-Nav-Item.jsx';
@@ -90,6 +91,8 @@ export default class FCCNav extends React.Component {
     } = this.props;
 
     if (isDropdown) {
+      // adding a noop to NavDropdown to disable false warning
+      // about controlled component
       return (
         <NavDropdown
           id={ `nav-${content}-dropdown` }
@@ -99,6 +102,7 @@ export default class FCCNav extends React.Component {
           onClose={ closeDropdown }
           onMouseEnter={ openDropdown }
           onMouseLeave={ closeDropdown }
+          onToggle={ noop }
           open={ isNavDropdownOpen }
           title={ content }
           >