From 7dd15740a0bc4986ee0dbcd224d20b978ff5eeca Mon Sep 17 00:00:00 2001
From: Mikhail Mikheev <mdoublemmm@gmail.com>
Date: Mon, 17 Jun 2019 14:35:58 +0400
Subject: [PATCH] add active styles for settings menu

---
 .../components/Settings/Threshold/index.jsx   |  9 ++++
 src/routes/safe/components/Settings/index.jsx | 45 ++++++++++---------
 src/routes/safe/components/Settings/style.js  |  7 ++-
 3 files changed, 39 insertions(+), 22 deletions(-)
 create mode 100644 src/routes/safe/components/Settings/Threshold/index.jsx

diff --git a/src/routes/safe/components/Settings/Threshold/index.jsx b/src/routes/safe/components/Settings/Threshold/index.jsx
new file mode 100644
index 00000000..c354bd56
--- /dev/null
+++ b/src/routes/safe/components/Settings/Threshold/index.jsx
@@ -0,0 +1,9 @@
+import React from 'react'
+
+const Threshold = () => {
+  return <div>
+    This will change the threshold
+  </div>
+}
+
+export default Threshold
\ No newline at end of file
diff --git a/src/routes/safe/components/Settings/index.jsx b/src/routes/safe/components/Settings/index.jsx
index 0b1360e8..d89a11a5 100644
--- a/src/routes/safe/components/Settings/index.jsx
+++ b/src/routes/safe/components/Settings/index.jsx
@@ -1,5 +1,6 @@
 // @flow
 import * as React from 'react'
+import cn from 'classnames'
 import { withStyles } from '@material-ui/core/styles'
 import Block from '~/components/layout/Block'
 import Col from '~/components/layout/Col'
@@ -7,10 +8,12 @@ import Row from '~/components/layout/Row'
 import RemoveSafeModal from './RemoveSafeModal'
 import Paragraph from '~/components/layout/Paragraph'
 import Hairline from '~/components/layout/Hairline'
+import Threshold from './Threshold'
 import { styles } from './style'
 
 type State = {
   showRemoveSafe: boolean,
+  menuOptionIndex: number,
 }
 
 type Props = {
@@ -44,11 +47,7 @@ class Settings extends React.Component<Props, State> {
   render() {
     const { showRemoveSafe, menuOptionIndex } = this.state
     const {
-      classes,
-      granted,
-      etherScanLink,
-      safeAddress,
-      safeName,
+      classes, granted, etherScanLink, safeAddress, safeName,
     } = this.props
 
     return (
@@ -73,21 +72,33 @@ class Settings extends React.Component<Props, State> {
         <Block className={classes.root}>
           <Col xs={3} layout="column">
             <Block className={classes.menu}>
-              <Row className={classes.menuOption} onClick={this.handleChange(1)}>
+              <Row
+                className={cn(classes.menuOption, menuOptionIndex === 1 && classes.active)}
+                onClick={this.handleChange(1)}
+              >
                 Safe name
               </Row>
               <Hairline />
               {granted && (
                 <React.Fragment>
-                  <Row className={classes.menuOption} onClick={this.handleChange(2)}>
+                  <Row
+                    className={cn(classes.menuOption, menuOptionIndex === 2 && classes.active)}
+                    onClick={this.handleChange(2)}
+                  >
                     Owners
                   </Row>
                   <Hairline />
-                  <Row className={classes.menuOption} onClick={this.handleChange(3)}>
+                  <Row
+                    className={cn(classes.menuOption, menuOptionIndex === 3 && classes.active)}
+                    onClick={this.handleChange(3)}
+                  >
                     Required confirmations
                   </Row>
                   <Hairline />
-                  <Row className={classes.menuOption} onClick={this.handleChange(4)}>
+                  <Row
+                    className={cn(classes.menuOption, menuOptionIndex === 4 && classes.active)}
+                    onClick={this.handleChange(4)}
+                  >
                     Modules
                   </Row>
                   <Hairline />
@@ -97,18 +108,10 @@ class Settings extends React.Component<Props, State> {
           </Col>
           <Col xs={9} layout="column">
             <Block className={classes.container}>
-              {menuOptionIndex === 1 && (
-                <p>To be done</p>
-              )}
-              {granted && menuOptionIndex === 2 && (
-                <p>To be done</p>
-              )}
-              {granted && menuOptionIndex === 3 && (
-                <p>To be done</p>
-              )}
-              {granted && menuOptionIndex === 4 && (
-                <p>To be done</p>
-              )}
+              {menuOptionIndex === 1 && <p>To be done</p>}
+              {granted && menuOptionIndex === 2 && <p>To be done</p>}
+              {granted && menuOptionIndex === 3 && <Threshold />}
+              {granted && menuOptionIndex === 4 && <p>To be done</p>}
             </Block>
           </Col>
         </Block>
diff --git a/src/routes/safe/components/Settings/style.js b/src/routes/safe/components/Settings/style.js
index 7fb64f13..ce8eac3c 100644
--- a/src/routes/safe/components/Settings/style.js
+++ b/src/routes/safe/components/Settings/style.js
@@ -1,6 +1,6 @@
 // @flow
 import {
-  sm, md, lg, border,
+  sm, md, lg, border, secondary, bolderFont,
 } from '~/theme/variables'
 
 export const styles = (theme: Object) => ({
@@ -22,6 +22,11 @@ export const styles = (theme: Object) => ({
     alignItems: 'center',
     cursor: 'pointer',
   },
+  active: {
+    backgroundColor: '#f4f4f9',
+    color: secondary,
+    fontWeight: bolderFont,
+  },
   container: {
     padding: lg,
   },