From b8aec634139e4ef2ca900c50ee1265034d017542 Mon Sep 17 00:00:00 2001
From: RachelElysia <71795832+RachelElysia@users.noreply.github.com>
Date: Mon, 3 May 2021 18:58:23 -0400
Subject: [PATCH] Host details page styling (#710)

* Host details page restyled to match Figma
* Page uses flexboxes
* Text match Figma
---
 assets/images/icon-chevron-down-9x6@2x.png    | Bin 0 -> 408 bytes
 .../hosts/HostDetailsPage/HostDetailsPage.jsx |  55 ++++++++++--------
 .../pages/hosts/HostDetailsPage/_styles.scss  |  50 ++++++++++++----
 3 files changed, 71 insertions(+), 34 deletions(-)
 create mode 100644 assets/images/icon-chevron-down-9x6@2x.png

diff --git a/assets/images/icon-chevron-down-9x6@2x.png b/assets/images/icon-chevron-down-9x6@2x.png
new file mode 100644
index 0000000000000000000000000000000000000000..46c0151dfe773ed186915a143068ba7b8b261fa7
GIT binary patch
literal 408
zcmV;J0cZY+P)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F800009a7bBm000&x
z000&x0ZCFM@Bjb+0drDELIAGL9O(c600d`2O+f$vv5yP<VFdsH0WC>HK~#7F?Uv6`
zgHRC0cLVB`QqqDQm<oc2sEG%$0<i<VU^EA@13O3ymXecB65$h&L*_wd_WhaU@_sjV
zSiX;CcOOI^kH>Rlm>e;?!2<#Dw)`C}7N=j*JSPTP@wB~MRVL1*CI)1QB3=`n48o-}
z-^xIb_&;3UlMv%iFuTS(5#sI>%gbyY<|M?Z0=?p(m9BV=b_Mo{Zy|5IT|()NNrGto
z-H2bf64x@Y`N5KzhAzcx#65s&zB@q&TSRQ&Y|SHpK9z7Y^OMPDNQP*?!}O}inR%>)
zwYJ|b+GF_g7Q_HPlyFmt+bPDRiP0XYO;la+tb`+!$aPH^uj1@ANQm*M#&u0todwxE
zNPA5bcp@Rzr(wrLyBnnW4u43BluEv{T}&R&!TAFkjF}8CSsNk%0000<MNUMnLSTX;
C6R1J}

literal 0
HcmV?d00001

diff --git a/frontend/pages/hosts/HostDetailsPage/HostDetailsPage.jsx b/frontend/pages/hosts/HostDetailsPage/HostDetailsPage.jsx
index 948b20846e..98a989fcf1 100644
--- a/frontend/pages/hosts/HostDetailsPage/HostDetailsPage.jsx
+++ b/frontend/pages/hosts/HostDetailsPage/HostDetailsPage.jsx
@@ -3,6 +3,7 @@ import PropTypes from "prop-types";
 import { connect } from "react-redux";
 import classnames from "classnames";
 
+import { Link } from "react-router";
 import ReactTooltip from "react-tooltip";
 import { noop, pick } from "lodash";
 
@@ -29,6 +30,8 @@ import {
 import helpers from "./helpers";
 import SelectQueryModal from "./SelectQueryModal";
 
+import BackChevron from "../../../../assets/images/icon-chevron-down-9x6@2x.png";
+
 const baseClass = "host-details";
 
 export class HostDetailsPage extends Component {
@@ -192,7 +195,7 @@ export class HostDetailsPage extends Component {
             You can’t <br /> query an <br /> offline host.
           </span>
         </ReactTooltip>
-        <Button onClick={toggleDeleteHostModal()} variant="inverse">
+        <Button onClick={toggleDeleteHostModal()} variant="active">
           Delete
         </Button>
       </div>
@@ -219,7 +222,7 @@ export class HostDetailsPage extends Component {
     });
 
     return (
-      <div className="section labels">
+      <div className="section labels col-25">
         <p className="section__header">Labels</p>
         {labels.length === 0 ? (
           <p className="info__item">No labels are associated with this host.</p>
@@ -344,6 +347,12 @@ export class HostDetailsPage extends Component {
       Object.keys(object).forEach((key) => {
         if (object[key] === "") {
           object[key] = "--";
+        } else if (
+          key === "logger_tls_period" ||
+          key === "config_tls_refresh" ||
+          key === "distributed_interval"
+        ) {
+          object[key] = `${object[key]} sec`;
         }
       });
     });
@@ -356,6 +365,12 @@ export class HostDetailsPage extends Component {
 
     return (
       <div className={`${baseClass} body-wrap`}>
+        <div>
+          <Link to="/hosts/manage">
+            <img src={BackChevron} alt="back chevron" id="back-chevron" />
+            Back to Hosts
+          </Link>
+        </div>
         <div className="section title">
           <div className="title__inner">
             <div className="hostname-container">
@@ -395,22 +410,20 @@ export class HostDetailsPage extends Component {
           </div>
           {renderActionButtons()}
         </div>
-        <div className="section about">
+        <div className="section about col-50">
           <p className="section__header">About this host</p>
           <div className="info">
             <div className="info__item info__item--about">
               <div className="info__block">
-                <span className="info__header">Last seen</span>
-                <span className="info__header">Enrolled</span>
-                <span className="info__header">Uptime</span>
-              </div>
-              <div className="info__block">
-                <span className="info__data">
-                  {humanHostLastSeen(aboutData.seen_time)}
-                </span>
+                <span className="info__header">Created at</span>
                 <span className="info__data">
                   {humanHostEnrolled(aboutData.last_enrolled_at)}
                 </span>
+                <span className="info__header">Updated at</span>
+                <span className="info__data">
+                  {humanHostLastSeen(aboutData.seen_time)}
+                </span>
+                <span className="info__header">Uptime</span>
                 <span className="info__data">
                   {humanHostUptime(aboutData.uptime)}
                 </span>
@@ -419,33 +432,27 @@ export class HostDetailsPage extends Component {
             <div className="info__item info__item--about">
               <div className="info__block">
                 <span className="info__header">Hardware model</span>
-                <span className="info__header">Serial number</span>
-                <span className="info__header">IP address</span>
-              </div>
-              <div className="info__block">
                 <span className="info__data">{aboutData.hardware_model}</span>
+                <span className="info__header">Serial number</span>
                 <span className="info__data">{aboutData.hardware_serial}</span>
+                <span className="info__header">IPv4</span>
                 <span className="info__data">{aboutData.primary_ip}</span>
               </div>
             </div>
           </div>
         </div>
-        <div className="section osquery">
-          <p className="section__header">Osquery configuration</p>
-          <div className="info">
-            <div className="info__item info__item--title">
-              <span className="info__header">Config refresh</span>
+        <div className="section osquery col-25">
+          <p className="section__header">Agent Options</p>
+          <div className="info__item info__item--about">
+            <div className="info__block">
+              <span className="info__header">Config TLS refresh</span>
               <span className="info__data">
                 {osqueryData.config_tls_refresh}
               </span>
-            </div>
-            <div className="info__item info__item--title">
               <span className="info__header">Logger TLS period</span>
               <span className="info__data">
                 {osqueryData.logger_tls_period}
               </span>
-            </div>
-            <div className="info__item info__item--title">
               <span className="info__header">Distributed interval</span>
               <span className="info__data">
                 {osqueryData.distributed_interval}
diff --git a/frontend/pages/hosts/HostDetailsPage/_styles.scss b/frontend/pages/hosts/HostDetailsPage/_styles.scss
index 126722c77e..f56ed3adba 100644
--- a/frontend/pages/hosts/HostDetailsPage/_styles.scss
+++ b/frontend/pages/hosts/HostDetailsPage/_styles.scss
@@ -1,17 +1,34 @@
+a {
+  font-size: $x-small;
+  color: $core-vibrant-blue;
+  font-weight: $bold;
+  text-decoration: none;
+}
+
 .host-details {
   display: flex;
-  flex-direction: column;
+  flex-wrap: wrap;
+  flex-grow: 1;
+  align-content: flex-start;
   padding-bottom: 50px;
   min-width: 0;
+  background-color: $ui-off-white;
+  gap: $pad-large;
 
   .section {
+    flex: 100%;
     display: flex;
     flex-direction: column;
-    margin: $pad-xxlarge 0 0;
+    background-color: $core-white;
+    border-radius: 16px;
+    border: 1px solid $ui-fleet-blue-15;
+    padding: $pad-xxlarge;
+    box-shadow: 0px 3px 0px rgba(226, 228, 234, 0.4);
 
     &__header {
-      margin: 0 0 $pad-medium;
       font-size: $medium;
+      font-weight: bold;
+      margin: 0 0 $pad-medium 0;
     }
 
     .info {
@@ -94,7 +111,8 @@
     }
   }
 
-  .about {
+  .about,
+  .osquery {
     .info {
       &__item {
         &--about {
@@ -107,18 +125,25 @@
         display: flex;
         flex-direction: column;
         margin-right: $pad-xxlarge;
+      }
+      &__data {
+        margin-bottom: $pad-medium;
 
-        span {
-          margin-bottom: $pad-medium;
-
-          &:last-child {
-            margin-bottom: 0;
-          }
+        &:last-child {
+          margin-bottom: 0;
         }
       }
     }
   }
 
+  .col-50 {
+    flex: 2;
+  }
+
+  .col-25 {
+    flex: 1;
+  }
+
   .status {
     color: $core-fleet-black;
     text-transform: capitalize;
@@ -224,4 +249,9 @@
       white-space: nowrap;
     }
   }
+
+  #back-chevron {
+    width: 12px;
+    margin-right: 10px;
+  }
 }