From 77f30df8c1d7f2fe342e5d1b011fed4dd962d839 Mon Sep 17 00:00:00 2001
From: meriadec <meriadec.pillet@gmail.com>
Date: Wed, 27 Jun 2018 16:38:39 +0200
Subject: [PATCH] Cursor `not-allowed` on disabled sidebar items

---
 src/components/base/SideBar/SideBarListItem.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/components/base/SideBar/SideBarListItem.js b/src/components/base/SideBar/SideBarListItem.js
index 671dee6f..d241fa3c 100644
--- a/src/components/base/SideBar/SideBarListItem.js
+++ b/src/components/base/SideBar/SideBarListItem.js
@@ -33,7 +33,7 @@ class SideBarListItem extends PureComponent<Props> {
       <Container
         isActive={!disabled && isActive}
         iconActiveColor={iconActiveColor}
-        onClick={disabled ? null : onClick}
+        onClick={disabled ? undefined : onClick}
         disabled={disabled}
       >
         {!!Icon && <Icon size={16} />}
@@ -62,8 +62,7 @@ const Container = styled(Tabbable).attrs({
   px: 3,
   py: 2,
 })`
-  cursor: ${p => (p.disabled || p.isActive ? 'default' : 'pointer')};
-  pointer-events: ${p => (p.isDisabled ? 'none' : 'auto')};
+  cursor: ${p => (p.disabled ? 'not-allowed' : p.isActive ? 'default' : 'pointer')};
   color: ${p => (p.isActive ? p.theme.colors.dark : p.theme.colors.smoke)};
   background: ${p => (p.isActive ? p.theme.colors.lightGrey : '')};
   opacity: ${p => (p.disabled ? 0.5 : 1)};