From 51b2021e95a6e75643c5613bdfe4208fe16dfc3e Mon Sep 17 00:00:00 2001
From: shahzad <shahzad31comp@gmail.com>
Date: Thu, 28 Nov 2019 19:44:12 +0100
Subject: [PATCH] update expand row in ping list

---
 .../__tests__/__snapshots__/monitor_list.test.tsx.snap      | 4 ++--
 .../__snapshots__/monitor_list_pagination.test.tsx.snap     | 4 ++--
 .../components/functional/monitor_list/monitor_list.tsx     | 2 +-
 .../__tests__/__snapshots__/ping_list.test.tsx.snap         | 6 ++++--
 .../public/components/functional/ping_list/ping_list.tsx    | 6 ++++--
 5 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list.test.tsx.snap b/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list.test.tsx.snap
index c2167993b80b93..f7afed7be4c1e1 100644
--- a/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list.test.tsx.snap
+++ b/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list.test.tsx.snap
@@ -67,7 +67,7 @@ exports[`MonitorList component renders a no items message when no data is provid
             "name": "",
             "render": [Function],
             "sortable": true,
-            "width": "40px",
+            "width": "24px",
           },
         ]
       }
@@ -172,7 +172,7 @@ exports[`MonitorList component renders the monitor list 1`] = `
             "name": "",
             "render": [Function],
             "sortable": true,
-            "width": "40px",
+            "width": "24px",
           },
         ]
       }
diff --git a/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list_pagination.test.tsx.snap b/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list_pagination.test.tsx.snap
index 619cce88b309ba..cea31539917d57 100644
--- a/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list_pagination.test.tsx.snap
+++ b/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/__tests__/__snapshots__/monitor_list_pagination.test.tsx.snap
@@ -67,7 +67,7 @@ exports[`MonitorList component renders a no items message when no data is provid
             "name": "",
             "render": [Function],
             "sortable": true,
-            "width": "40px",
+            "width": "24px",
           },
         ]
       }
@@ -172,7 +172,7 @@ exports[`MonitorList component renders the monitor list 1`] = `
             "name": "",
             "render": [Function],
             "sortable": true,
-            "width": "40px",
+            "width": "24px",
           },
         ]
       }
diff --git a/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/monitor_list.tsx b/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/monitor_list.tsx
index 274a7ab0be9be4..c94e39247c44ba 100644
--- a/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/monitor_list.tsx
+++ b/x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/monitor_list.tsx
@@ -208,7 +208,7 @@ export const MonitorListComponent = (props: Props) => {
               field: 'monitor_id',
               name: '',
               sortable: true,
-              width: '40px',
+              width: '24px',
               isExpander: true,
               render: (id: string) => {
                 return (
diff --git a/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/__tests__/__snapshots__/ping_list.test.tsx.snap b/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/__tests__/__snapshots__/ping_list.test.tsx.snap
index 5f60ce38500c80..a2e31a0f33d39f 100644
--- a/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/__tests__/__snapshots__/ping_list.test.tsx.snap
+++ b/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/__tests__/__snapshots__/ping_list.test.tsx.snap
@@ -145,7 +145,7 @@ exports[`PingList component renders sorted list without errors 1`] = `
             "render": [Function],
           },
           Object {
-            "align": "right",
+            "align": "center",
             "field": "http.response.status_code",
             "name": "Response code",
             "render": [Function],
@@ -154,10 +154,12 @@ exports[`PingList component renders sorted list without errors 1`] = `
             "align": "right",
             "isExpander": true,
             "render": [Function],
-            "width": "40px",
+            "width": "24px",
           },
         ]
       }
+      hasActions={true}
+      isExpandable={true}
       itemId="id"
       itemIdToExpandedRowMap={Object {}}
       items={
diff --git a/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/ping_list.tsx b/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/ping_list.tsx
index 0a97b596a7a71e..b3c1de37c13406 100644
--- a/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/ping_list.tsx
+++ b/x-pack/legacy/plugins/uptime/public/components/functional/ping_list/ping_list.tsx
@@ -190,7 +190,7 @@ export const PingListComponent = ({
   if (hasStatus) {
     columns.push({
       field: 'http.response.status_code',
-      align: 'right',
+      align: 'center',
       name: i18n.translate('xpack.uptime.pingList.responseCodeColumnLabel', {
         defaultMessage: 'Response code',
       }),
@@ -200,7 +200,7 @@ export const PingListComponent = ({
 
   columns.push({
     align: 'right',
-    width: '40px',
+    width: '24px',
     isExpander: true,
     render: (item: Ping) => (
       <EuiButtonIcon
@@ -300,6 +300,8 @@ export const PingListComponent = ({
         <EuiBasicTable
           loading={loading}
           columns={columns}
+          isExpandable={true}
+          hasActions={true}
           items={pings}
           itemId="id"
           itemIdToExpandedRowMap={itemIdToExpandedRowMap}