From 2095eaf00c590f63f5dd908ea254a1aeca8eab12 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Se=CC=81rgio=20Ramos?= <mail@sergioramos.me>
Date: Wed, 11 Jan 2017 17:42:47 +0000
Subject: [PATCH] recursive service item with metrics

---
 frontend/src/components/service-item/index.js | 126 +++++++++++++-----
 ui/src/components/list/transfer-props.js      |   2 +-
 2 files changed, 92 insertions(+), 36 deletions(-)

diff --git a/frontend/src/components/service-item/index.js b/frontend/src/components/service-item/index.js
index 2680f894..c3e6e528 100644
--- a/frontend/src/components/service-item/index.js
+++ b/frontend/src/components/service-item/index.js
@@ -1,14 +1,23 @@
+const forceArray = require('force-array');
 const React = require('react');
 const ReactRouter = require('react-router');
+const Styled = require('styled-components');
 
 const Anchor = require('@ui/components/anchor');
+const Column = require('@ui/components/column');
 const List = require('@ui/components/list');
+const MiniMetric = require('@ui/components/mini-metric');
 const PropTypes = require('@root/prop-types');
+const Row = require('@ui/components/row');
 
 const {
   Link
 } = ReactRouter;
 
+const {
+  default: styled
+} = Styled;
+
 const {
   ListItem,
   ListItemView,
@@ -22,30 +31,84 @@ const {
   ListItemHeader
 } = List;
 
+const {
+  MiniMetricGraph,
+  MiniMetricMeta,
+  MiniMetricTitle,
+  MiniMetricSubtitle,
+  MiniMetricView
+} = MiniMetric;
+
+const MetricsRow = styled(Row)`
+  margin: 0;
+
+  & > div {
+    padding-left: 0;
+    padding-right: 0;
+  }
+`;
+
 const ServiceItem = ({
   org = '',
   project = '',
   service = {}
 }) => {
+  const isChild = !!service.parent;
+
+  const childs = forceArray(service.services).map((service) => (
+    <ServiceItem
+      key={service.uuid}
+      org={org}
+      project={project}
+      service={service}
+    />
+  ));
+
   const to = `/${org}/projects/${project}/services/${service.id}`;
 
-  const childs = service.services.map((service) => (
-    <ListItem
-      collapsed={service.collapsed}
-      flat
-      key={service.uuid}
-      stacked={service.instances > 1}
-    >
-      <ListItemView>
-        <ListItemMeta>
-          <ListItemTitle>{service.name}</ListItemTitle>
-          <ListItemSubTitle>{service.instances} instances</ListItemSubTitle>
-        </ListItemMeta>
-        <ListItemOutlet>
-          Metrics
-        </ListItemOutlet>
-      </ListItemView>
-    </ListItem>
+  const title = isChild ? (
+    <ListItemTitle>{service.name}</ListItemTitle>
+  ) : (
+    <ListItemTitle>
+      <Link to={to}>
+        {Anchor.fn(
+          <Anchor secondary>
+            {service.name}
+          </Anchor>
+        )}
+      </Link>
+    </ListItemTitle>
+  );
+
+  const subtitle = (
+    <ListItemSubTitle>{service.instances} instances</ListItemSubTitle>
+  );
+
+  const description = (
+    <ListItemDescription>Flags</ListItemDescription>
+  );
+
+  const header = isChild ? null : (
+    <ListItemHeader>
+      <ListItemMeta>
+        {title}
+        {subtitle}
+        {description}
+      </ListItemMeta>
+      <ListItemOptions>…</ListItemOptions>
+    </ListItemHeader>
+  );
+
+  const metrics = service.metrics.map((metric, i) => (
+    <Column key={i} xs={4}>
+      <MiniMetricView borderless>
+        <MiniMetricMeta>
+          <MiniMetricTitle>Memory: 54%</MiniMetricTitle>
+          <MiniMetricSubtitle>(1280/3000 MB)</MiniMetricSubtitle>
+        </MiniMetricMeta>
+        <MiniMetricGraph data={metric.data} />
+      </MiniMetricView>
+    </Column>
   ));
 
   const view = childs.length ? (
@@ -55,10 +118,14 @@ const ServiceItem = ({
   ) : (
     <ListItemView>
       <ListItemMeta>
-        <ListItemDescription>Flags</ListItemDescription>
+        {isChild && title}
+        {isChild && subtitle}
+        {description}
       </ListItemMeta>
       <ListItemOutlet>
-        Metrics
+        <MetricsRow>
+          {metrics}
+        </MetricsRow>
       </ListItemOutlet>
     </ListItemView>
   );
@@ -66,23 +133,12 @@ const ServiceItem = ({
   return (
     <ListItem
       collapsed={service.collapsed}
-      headed
+      flat={isChild}
+      headed={!isChild}
+      key={service.uuid}
+      stacked={isChild && (service.instances > 1)}
     >
-      <ListItemHeader>
-        <ListItemMeta>
-          <ListItemTitle>
-            <Link to={to}>
-              {Anchor.fn(
-                <Anchor secondary>
-                  {service.name}
-                </Anchor>
-              )}
-            </Link>
-          </ListItemTitle>
-          <ListItemSubTitle>{service.instances} instance</ListItemSubTitle>
-        </ListItemMeta>
-        <ListItemOptions>…</ListItemOptions>
-      </ListItemHeader>
+      {header}
       {view}
     </ListItem>
   );
diff --git a/ui/src/components/list/transfer-props.js b/ui/src/components/list/transfer-props.js
index 0d8f4284..275dc6e3 100644
--- a/ui/src/components/list/transfer-props.js
+++ b/ui/src/components/list/transfer-props.js
@@ -4,7 +4,7 @@ const React = require('react');
 const transfer = (parentProps, props) => {
 // eslint-disable-next-line react/prop-types
   return React.Children.map(props.children, (c) => {
-    return React.cloneElement(c, {
+    return c && React.cloneElement(c, {
       ...c.props,
       ...parentProps.reduce((sum, name) => ({
         ...sum,