From 5057a901fe975f2912e73e8a4bd32d2cf86513f5 Mon Sep 17 00:00:00 2001
From: Cassie Tarakajian <ctarakajian@gmail.com>
Date: Fri, 4 May 2018 17:43:31 -0700
Subject: [PATCH] fix a few linting errors

---
 .eslintrc                                   | 13 ++++++++++++-
 client/components/Nav.jsx                   |  2 +-
 client/modules/App/components/DevTools.jsx  | 16 ++++++++++------
 client/modules/IDE/components/AssetList.jsx |  2 +-
 client/modules/IDE/components/Console.jsx   |  3 +--
 5 files changed, 25 insertions(+), 11 deletions(-)

diff --git a/.eslintrc b/.eslintrc
index 6b66fa55..7be0a9eb 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -40,7 +40,18 @@
     }],
     "class-methods-use-this": 0,
     "react/jsx-no-bind": [2, {"allowBind": true, "allowArrowFunctions": true}],
-    "no-return-assign": [2, "except-parens"]
+    "no-return-assign": [2, "except-parens"],
+    "jsx-a11y/anchor-is-valid": [
+      "error",
+      {
+        "components": [
+          "Link"
+        ],
+        "specialLink": [
+          "to"
+        ]
+      }
+    ]
   },
   "plugins": [
     "react", "jsx-a11y", "import"
diff --git a/client/components/Nav.jsx b/client/components/Nav.jsx
index beef9d41..09b3c123 100644
--- a/client/components/Nav.jsx
+++ b/client/components/Nav.jsx
@@ -108,7 +108,7 @@ class Nav extends React.PureComponent {
       })
     };
     return (
-      <nav className="nav" role="navigation" title="main-navigation" ref={(node) => { this.node = node; }}>
+      <nav className="nav" title="main-navigation" ref={(node) => { this.node = node; }}>
         <ul className="nav__items-left" title="project-menu">
           <li className="nav__item-logo">
             <InlineSVG src={logoUrl} alt="p5.js logo" />
diff --git a/client/modules/App/components/DevTools.jsx b/client/modules/App/components/DevTools.jsx
index 0f5d2f2e..32f987d5 100644
--- a/client/modules/App/components/DevTools.jsx
+++ b/client/modules/App/components/DevTools.jsx
@@ -3,9 +3,13 @@ import { createDevTools } from 'redux-devtools';
 import LogMonitor from 'redux-devtools-log-monitor';
 import DockMonitor from 'redux-devtools-dock-monitor';
 
-export default createDevTools(<DockMonitor
-  toggleVisibilityKey="ctrl-h"
-  changePositionKey="ctrl-w"
->
-  <LogMonitor />
-</DockMonitor>);
+const devTools = (
+  <DockMonitor
+    toggleVisibilityKey="ctrl-h"
+    changePositionKey="ctrl-w"
+  >
+    <LogMonitor />
+  </DockMonitor>
+);
+
+export default createDevTools(devTools);
diff --git a/client/modules/IDE/components/AssetList.jsx b/client/modules/IDE/components/AssetList.jsx
index 85709385..025d6e4f 100644
--- a/client/modules/IDE/components/AssetList.jsx
+++ b/client/modules/IDE/components/AssetList.jsx
@@ -49,7 +49,7 @@ class AssetList extends React.Component {
                   <td>{prettyBytes(asset.size)}</td>
                   <td><Link to={asset.url} target="_blank">View</Link></td>
                   <td><Link to={`/${this.props.username}/sketches/${asset.sketchId}`}>{asset.sketchName}</Link></td>
-                 </tr>))}
+                </tr>))}
             </tbody>
           </table>}
       </div>
diff --git a/client/modules/IDE/components/Console.jsx b/client/modules/IDE/components/Console.jsx
index ee8c43f7..14d6c6b6 100644
--- a/client/modules/IDE/components/Console.jsx
+++ b/client/modules/IDE/components/Console.jsx
@@ -39,8 +39,7 @@ class Console extends React.Component {
         </div>
         <div ref={(element) => { this.consoleMessages = element; }} className="preview-console__messages">
           {this.props.consoleEvents.map((consoleEvent) => {
-            const args = consoleEvent.arguments;
-            const method = consoleEvent.method;
+            const { args, method } = consoleEvent;
             if (Object.keys(args).length === 0) {
               return (
                 <div key={consoleEvent.id} className="preview-console__undefined">