From 5219b97601d3aee849081f81536f8edd91d9ad53 Mon Sep 17 00:00:00 2001
From: Janicki Artur <artur.janicki@contractors.roche.com>
Date: Wed, 18 Jul 2018 14:55:43 +0200
Subject: [PATCH 1/1] clean code for single-list-item

Feature #13797

Arvados-DCO-1.1-Signed-off-by: Janicki Artur <artur.janicki@contractors.roche.com>
---
 .../single-list-item/single-list-item.tsx     | 26 +++++--------------
 1 file changed, 7 insertions(+), 19 deletions(-)

diff --git a/src/components/single-list-item/single-list-item.tsx b/src/components/single-list-item/single-list-item.tsx
index 13338c3d..17bf8b58 100644
--- a/src/components/single-list-item/single-list-item.tsx
+++ b/src/components/single-list-item/single-list-item.tsx
@@ -23,34 +23,22 @@ class SingleListItem extends React.Component<SingleListItemProps, {}> {
         const { classes, isActive, hasMargin, name, icon: Icon } = this.props;
         return (
             <Typography component='span' className={classes.root}>
-                <ListItemIcon className={this.getListItemIconClassNames(hasMargin, isActive)}>
+                <ListItemIcon className={classnames({
+                        [classes.hasMargin]: hasMargin,
+                        [classes.active]: isActive
+                    })}>
                     <Icon />
                 </ListItemIcon>
                 <ListItemText primary={
-                    <Typography variant='body1' className={this.getListItemTextClassNames(isActive)}>
+                    <Typography variant='body1' className={classnames(classes.listItemText, {
+                            [classes.active]: isActive
+                        })}>
                         {name}
                     </Typography>
                 } />
             </Typography>
         );
     }
-
-    getListItemIconClassNames = (hasMargin?: boolean, isActive?: boolean) => {
-        const { classes } = this.props;
-        return classnames({
-            [classes.hasMargin]: hasMargin,
-            [classes.active]: isActive
-        });
-    }
-
-    getListItemTextClassNames = (isActive?: boolean) => {
-        const { classes } = this.props;
-        return classnames(classes.listItemText, {
-            [classes.active]: isActive
-        });
-    }
-
-
 }
         
 type CssRules = 'root' | 'listItemText' | 'hasMargin' | 'active';
-- 
2.30.2