[Uptime] Fix jerky monitor list expanded row behavior (#47080) (#47113)

* [Uptime] Fix jerky monitor list expanded row behavior

Only set loading to true when there are no items present to prevent the bug outlined in
in elastic/eui#2393 . Once that is fixed we can simply set the value here to
loading={loading}

Fixes #46244

* Also fix loading state of charts to not move the table around

x-pack/legacy/plugins/uptime/public/components/functional/charts/chart_wrapper/chart_wrapper.tsx

@@ -30,7 +30,7 @@ export const ChartWrapper: FC<Props> = ({ loading = false, height = '100%', chil
         <EuiFlexGroup
           justifyContent="spaceAround"
           alignItems="center"
-          style={{ height, marginTop: `-${height}` }}
+          style={{ height, marginTop: `-${height}`, marginBottom: 0 }}
         >
           <EuiFlexItem grow={false}>
             <EuiLoadingChart size="xl" />

x-pack/legacy/plugins/uptime/public/components/functional/monitor_list/monitor_list.tsx

@@ -104,7 +104,10 @@ export const MonitorListComponent = (props: Props) => {
         <EuiSpacer size="s" />
         <EuiBasicTable
           error={errors ? formatUptimeGraphQLErrorList(errors) : errors}
-          loading={loading}
+          // Only set loading to true when there are no items present to prevent the bug outlined in
+          // in https://github.com/elastic/eui/issues/2393 . Once that is fixed we can simply set the value here to
+          // loading={loading}
+          loading={loading && (!items || items.length < 1)}
           isExpandable={true}
           hasActions={true}
           itemId="monitor_id"