From 4a0b05ded49ab0824dfa344a9274b13c2693c8ed Mon Sep 17 00:00:00 2001
From: apanizo <adolfo.panizo@gmail.com>
Date: Thu, 4 Oct 2018 11:16:03 +0200
Subject: [PATCH] Design of Review view of Open Safe process

---
 .../components/ReviewInformation/index.jsx    | 136 ++++++++++++++----
 1 file changed, 106 insertions(+), 30 deletions(-)

diff --git a/src/routes/open/components/ReviewInformation/index.jsx b/src/routes/open/components/ReviewInformation/index.jsx
index 343a5575..ccf71ebd 100644
--- a/src/routes/open/components/ReviewInformation/index.jsx
+++ b/src/routes/open/components/ReviewInformation/index.jsx
@@ -2,47 +2,123 @@
 import * as React from 'react'
 import { getNamesFrom, getAccountsFrom } from '~/routes/open/utils/safeDataExtractor'
 import Block from '~/components/layout/Block'
-import Bold from '~/components/layout/Bold'
+import { withStyles } from '@material-ui/core/styles'
+import Identicon from '~/components/Identicon'
 import OpenPaper from '~/components/Stepper/OpenPaper'
 import Col from '~/components/layout/Col'
-import Heading from '~/components/layout/Heading'
 import Row from '~/components/layout/Row'
 import Paragraph from '~/components/layout/Paragraph'
-import { FIELD_NAME, FIELD_CONFIRMATIONS, FIELD_DAILY_LIMIT } from '../fields'
+import { md, lg, border, background } from '~/theme/variables'
+import Hairline from '~/components/layout/Hairline'
+import { FIELD_NAME, FIELD_CONFIRMATIONS, getNumOwnersFrom } from '../fields'
 
-const ReviewInformation = () => (controls: React$Node, { values }: Object) => {
+const styles = () => ({
+  root: {
+    minHeight: '300px',
+  },
+  details: {
+    padding: lg,
+    borderRight: `solid 1px ${border}`,
+    height: '100%',
+  },
+  info: {
+    backgroundColor: background,
+    padding: lg,
+    justifyContent: 'center',
+  },
+  owners: {
+    padding: lg,
+  },
+  owner: {
+    padding: md,
+    alignItems: 'center',
+  },
+})
+
+type Props = {
+  values: Object,
+  classes: Object,
+}
+
+const ReviewComponent = ({ values, classes }: Props) => {
   const names = getNamesFrom(values)
   const addresses = getAccountsFrom(values)
+  const numOwners = getNumOwnersFrom(values)
 
   return (
-    <OpenPaper controls={controls}>
-      <Heading tag="h2">Review the Safe information</Heading>
-      <Paragraph>
-        <Bold>Safe Name: </Bold> {values[FIELD_NAME]}
-      </Paragraph>
-      <Paragraph>
-        <Bold>Required confirmations: </Bold> {values[FIELD_CONFIRMATIONS]}
-      </Paragraph>
-      <Paragraph>
-        <Bold>Daily limit: </Bold> {values[FIELD_DAILY_LIMIT]} ETH
-      </Paragraph>
-      <Heading tag="h3">Owners</Heading>
-      { names.map((name, index) => (
-        <Row key={`name${(index)}`} margin="md">
-          <Col xs={11} xsOffset={1}margin="sm">
-            <Block>
-              <Paragraph noMargin>{name}</Paragraph>
+    <React.Fragment>
+      <Row className={classes.root}>
+        <Col xs={4} layout="column">
+          <Block className={classes.details}>
+            <Block margin="lg">
+              <Paragraph size="lg" color="primary" noMargin>
+                Details
+              </Paragraph>
             </Block>
-          </Col>
-          <Col xs={11} xsOffset={1} margin="sm">
-            <Block>
-              <Paragraph noMargin>{addresses[index]}</Paragraph>
+            <Block margin="lg">
+              <Paragraph size="sm" color="disabled" noMargin>
+                Name of new Safe
+              </Paragraph>
+              <Paragraph size="lg" color="primary" noMargin weight="bolder">
+                {values[FIELD_NAME]}
+              </Paragraph>
             </Block>
-          </Col>
-        </Row>
-      ))}
-    </OpenPaper>
+            <Block margin="lg">
+              <Paragraph size="sm" color="disabled" noMargin>
+                Any transaction over the daily limit requires the confirmation of
+              </Paragraph>
+              <Paragraph size="lg" color="primary" noMargin weight="bolder">
+                {`${values[FIELD_CONFIRMATIONS]} out of ${numOwners} owners`}
+              </Paragraph>
+            </Block>
+          </Block>
+        </Col>
+        <Col xs={8} layout="column">
+          <Block className={classes.owners}>
+            <Paragraph size="lg" color="primary" noMargin>
+              {`${numOwners} Safe owners`}
+            </Paragraph>
+          </Block>
+          <Hairline />
+          { names.map((name, index) => (
+            <React.Fragment>
+              <Row key={`name${(index)}`} className={classes.owner}>
+                <Col xs={1} align="center">
+                  <Identicon address={addresses[index]} diameter={32} />
+                </Col>
+                <Col xs={11}>
+                  <Block>
+                    <Paragraph size="lg" noMargin>{name}</Paragraph>
+                    <Paragraph size="md" color="disabled" noMargin>{addresses[index]}</Paragraph>
+                  </Block>
+                </Col>
+              </Row>
+              <Hairline />
+            </React.Fragment>
+          ))}
+        </Col>
+      </Row>
+      <Row className={classes.info} align="center">
+        <Paragraph noMargin color="primary" size="md">
+          {'You\'re about to create a new Safe.'}
+        </Paragraph>
+        <Paragraph noMargin color="primary" size="md">
+          Make sure you have enough ETH in your wallet client to fund this transaction.
+        </Paragraph>
+      </Row>
+    </React.Fragment>
   )
 }
 
-export default ReviewInformation
+const ReviewPage = withStyles(styles)(ReviewComponent)
+
+const Review = () => (controls: React$Node, { values }: Object) => (
+  <React.Fragment>
+    <OpenPaper controls={controls} padding={false}>
+      <ReviewPage values={values} />
+    </OpenPaper>
+  </React.Fragment>
+)
+
+
+export default Review