From 261b4eddf3c1215f2e954779a485b290103109c1 Mon Sep 17 00:00:00 2001
From: Jonathan Rainville <rainville.jonathan@gmail.com>
Date: Fri, 19 Oct 2018 09:55:41 -0400
Subject: [PATCH] linting

---
 embark-ui/src/components/Layout.js    | 50 +++++++++++++--------------
 embark-ui/src/components/SearchBar.js |  2 +-
 embark-ui/src/components/Widget02.js  |  2 +-
 3 files changed, 26 insertions(+), 28 deletions(-)

diff --git a/embark-ui/src/components/Layout.js b/embark-ui/src/components/Layout.js
index e5bff8e5..15682182 100644
--- a/embark-ui/src/components/Layout.js
+++ b/embark-ui/src/components/Layout.js
@@ -1,9 +1,7 @@
 import React from 'react';
-import {Link} from 'react-router-dom';
+import {Link, withRouter} from 'react-router-dom';
 import PropTypes from 'prop-types';
-import {DropdownItem, DropdownMenu, DropdownToggle, Nav, NavItem, NavLink, Container} from 'reactstrap';
 import {connect} from 'react-redux';
-import {withRouter} from "react-router-dom";
 import {DropdownItem, DropdownMenu, DropdownToggle, Nav, NavItem, NavLink, Container, Alert} from 'reactstrap';
 import {explorerSearch} from "../actions";
 import {LIGHT_THEME, DARK_THEME} from '../constants';
@@ -55,12 +53,12 @@ const SIDEBAR_NAV_ITEMS = {
 const removeCssClasses = () => {
   document.body.classList.remove('sidebar-fixed');
   document.body.classList.remove('sidebar-lg-show');
-}
+};
 
 const getSidebar = (location) => {
   const currentItem = Object.keys(SIDEBAR_NAV_ITEMS).find(path => location.pathname.startsWith(path));
   return currentItem && SIDEBAR_NAV_ITEMS[currentItem];
-}
+};
 
 class Layout extends React.Component {
   constructor(props) {
@@ -114,11 +112,11 @@ class Layout extends React.Component {
 
   renderNav() {
     return (<Nav className="d-md-down-none" navbar>
-      {sidebarNavItems.items.map((item) => {
+      {HEADER_NAV_ITEMS.map((item) => {
         return (
-          <NavItem className="px-3" key={item.url}>
-            <NavLink href={item.url}>
-              <i className={item.icon}>&nbsp;</i>
+          <NavItem className="px-3" key={item.to}>
+            <NavLink exact tag={Link} to={item.to}>
+              <FontAwesome className="mr-2" name={item.icon} />
               {item.name}
             </NavLink>
           </NavItem>
@@ -155,16 +153,6 @@ class Layout extends React.Component {
     </AppHeaderDropdown>);
   }
 
-  renderSideBar() {
-    return (<AppSidebar fixed display="lg">
-      <AppSidebarHeader />
-      <AppSidebarForm />
-      <AppSidebarNav navConfig={sidebarNavItems} location={this.props.location} />
-      <AppSidebarFooter />
-      <AppSidebarMinimizer />
-    </AppSidebar>);
-  }
-
   renderFooter() {
     return (<AppFooter>
         <span className="ml-auto">
@@ -177,16 +165,18 @@ class Layout extends React.Component {
   }
 
   render() {
-    const {children, searchResult} = this.props;
+    const {children, searchResult, location} = this.props;
+    const sidebar = getSidebar(location);
+    if (!sidebar) {
+      removeCssClasses();
+    }
 
     return (<div className="app animated fadeIn">
       <AppHeader fixed>
-        <AppSidebarToggler className="d-lg-none" display="md" mobile />
-        <AppNavbarBrand
-          full={{ src: logo, width: 50, height: 50, alt: 'Embark Logo' }}
-          minimized={{ src: logo, width: 30, height: 30, alt: 'Embark Logo' }}
+        <AppNavbarBrand className="mx-3"
+                        full={{src: logo, width: 50, height: 50, alt: 'Embark Logo'}}
+                        minimized={{src: logo, width: 30, height: 30, alt: 'Embark Logo'}}
         />
-        <AppSidebarToggler className="d-md-down-none" display="lg" />
 
         {this.renderNav()}
 
@@ -194,7 +184,15 @@ class Layout extends React.Component {
       </AppHeader>
 
       <div className="app-body">
-        {this.renderSideBar()}
+        {sidebar &&
+        <AppSidebar fixed display="lg">
+          <AppSidebarHeader />
+          <AppSidebarForm />
+          <AppSidebarNav navConfig={sidebar} location={location} />
+          <AppSidebarFooter />
+          <AppSidebarMinimizer />
+        </AppSidebar>
+        }
 
         <main className="main">
           {searchResult.error && this.state.searchError && <Alert color="danger">{searchResult.error}
diff --git a/embark-ui/src/components/SearchBar.js b/embark-ui/src/components/SearchBar.js
index 4ab27dc6..3072019a 100644
--- a/embark-ui/src/components/SearchBar.js
+++ b/embark-ui/src/components/SearchBar.js
@@ -1,6 +1,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import {Form, FormGroup, Input, Button, Row, Col} from 'reactstrap';
+import {Form, FormGroup, Input, Button} from 'reactstrap';
 import FontAwesome from 'react-fontawesome';
 import classNames from 'classnames';
 
diff --git a/embark-ui/src/components/Widget02.js b/embark-ui/src/components/Widget02.js
index c5110d8c..6caf5ae8 100644
--- a/embark-ui/src/components/Widget02.js
+++ b/embark-ui/src/components/Widget02.js
@@ -1,6 +1,6 @@
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
-import { Card, CardBody, CardFooter } from 'reactstrap';
+import { Card, CardBody } from 'reactstrap';
 import classNames from 'classnames';
 import { mapToCssModules } from 'reactstrap/lib/utils';