From 87f2f5c7962e89412a4149519b0aba310179b4bd Mon Sep 17 00:00:00 2001
From: Lachlan Kermode <lachiekermode@gmail.com>
Date: Thu, 17 Jan 2019 10:41:53 +0000
Subject: [PATCH] fix sites toggle hover states

---
 src/components/ToolbarBottomActions.jsx          | 2 +-
 src/components/presentational/Icons/SitesIcon.js | 7 +++++--
 src/scss/toolbar.scss                            | 4 ++++
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/src/components/ToolbarBottomActions.jsx b/src/components/ToolbarBottomActions.jsx
index 7e20bd2..e20a4be 100644
--- a/src/components/ToolbarBottomActions.jsx
+++ b/src/components/ToolbarBottomActions.jsx
@@ -14,7 +14,7 @@ function ToolbarBottomActions (props) {
           {/*   isEnabled={this.props.viewFilters.routes} */}
           {/* /> */}
           <SitesIcon
-            isEnabled={props.sites.enabled}
+            isActive={props.sites.enabled}
             onClickHandler={props.sites.toggle}
           />
           {/* <CoeventIcon */}
diff --git a/src/components/presentational/Icons/SitesIcon.js b/src/components/presentational/Icons/SitesIcon.js
index 71a4ae4..226d31e 100644
--- a/src/components/presentational/Icons/SitesIcon.js
+++ b/src/components/presentational/Icons/SitesIcon.js
@@ -1,7 +1,10 @@
 import React from 'react';
 
-const SitesIcon = ({ isEnabled, onClickHandler }) => {
-  const classes = (isEnabled) ? 'action-button enabled' : 'action-button disabled';
+const SitesIcon = ({ isActive, isDisabled, onClickHandler }) => {
+  let classes = (isActive) ? 'action-button enabled' : 'action-button';
+  if (isDisabled) {
+    classes = 'action-button disabled'
+  }
 
   return (
     <button
diff --git a/src/scss/toolbar.scss b/src/scss/toolbar.scss
index 0d9d400..29a81ef 100644
--- a/src/scss/toolbar.scss
+++ b/src/scss/toolbar.scss
@@ -96,6 +96,10 @@
           }
         }
 
+        &:hover {
+          cursor: pointer;
+        }
+
         &:hover:not(.disabled) {
           transition: 0.2s ease;
           border: 1px solid $offwhite;