[EuiTables] Misc syntax cleanups/chores

- remove unnecessary props that already default to false

- various syntax nits

- small CSS fixes

src/plugins/unified_doc_viewer/public/components/doc_viewer_table/legacy/table.tsx

@@ -116,7 +116,6 @@ export const DocViewerLegacyTable = ({
       items={items}
       columns={tableColumns}
       rowProps={onSetRowProps}
-      pagination={false}
       responsiveBreakpoint={false}
     />
   );

x-pack/plugins/aiops/public/components/change_point_detection/change_points_table.tsx

@@ -313,7 +313,7 @@ export const ChangePointsTable: FC<ChangePointsTableProps> = ({
 
   return (
     <EuiInMemoryTable<ChangePointAnnotation>
-      itemId={'id'}
+      itemId="id"
       selection={selectionValue}
       loading={isLoading}
       data-test-subj={`aiopsChangePointResultsTable ${isLoading ? 'loading' : 'loaded'}`}

x-pack/plugins/data_visualizer/public/application/common/components/stats_table/data_visualizer_stats_table.tsx

@@ -395,7 +395,7 @@ export const DataVisualizerTable = <T extends DataVisualizerTableItem>({
       backgroundColor: euiTheme.colors.emptyShade,
       boxShadow: `inset 0 0px 0, inset 0 -1px 0 ${euiTheme.border.color}`,
     },
-    '.euiTableRow > .euiTableRowCel': {
+    '.euiTableRow > .euiTableRowCell': {
       borderTop: 0,
     },
     [useEuiMinBreakpoint('s')]: {

x-pack/plugins/observability_solution/synthetics/public/apps/synthetics/components/common/monitor_test_result/browser_steps_list.tsx

@@ -308,7 +308,7 @@ export const BrowserStepsList = ({
                 defaultMessage: 'No data found',
               })
         }
-        tableLayout={'auto'}
+        tableLayout="auto"
         itemId="_id"
         itemIdToExpandedRowMap={testNowMode || showExpand ? expandedMap : undefined}
       />

x-pack/plugins/observability_solution/synthetics/public/apps/synthetics/components/monitor_details/monitor_summary/test_runs_table.tsx

@@ -295,13 +295,12 @@ export const TestRunsTable = ({
         itemId="docId"
         itemIdToExpandedRowMap={expandedRows}
         css={{ overflowX: isTabletOrGreater ? 'auto' : undefined }}
-        compressed={false}
         loading={pingsLoading}
         columns={columns}
         error={pingsError?.body?.message}
         items={sortedPings}
         noItemsMessage={pingsLoading ? LOADING_TEST_RUNS : NO_DATA_FOUND}
-        tableLayout={'auto'}
+        tableLayout="auto"
         sorting={sorting}
         onChange={handleTableChange}
         rowProps={getRowProps}

x-pack/plugins/observability_solution/synthetics/public/apps/synthetics/components/monitors_page/management/monitor_list_table/monitor_list.tsx

@@ -11,8 +11,7 @@ import {
   EuiBasicTable,
   EuiTableSortingType,
   EuiPanel,
-  EuiSpacer,
-  useEuiTheme,
+  EuiHorizontalRule,
   useIsWithinMinBreakpoint,
 } from '@elastic/eui';
 import { i18n } from '@kbn/i18n';
@@ -50,7 +49,6 @@ export const MonitorList = ({
   loadPage,
   reloadPage,
 }: Props) => {
-  const { euiTheme } = useEuiTheme();
   const isXl = useIsWithinMinBreakpoint('xxl');
 
   const [monitorPendingDeletion, setMonitorPendingDeletion] =
@@ -104,8 +102,7 @@ export const MonitorList = ({
     <>
       <EuiPanel hasBorder={false} hasShadow={false} paddingSize="none">
         {recordRangeLabel}
-        <EuiSpacer size="s" />
-        <hr style={{ border: `1px solid ${euiTheme.colors.lightShade}` }} />
+        <EuiHorizontalRule margin="s" />
         <EuiBasicTable
           aria-label={i18n.translate('xpack.synthetics.management.monitorList.title', {
             defaultMessage: 'Synthetics monitors list',

x-pack/plugins/observability_solution/synthetics/public/apps/synthetics/components/step_details_page/step_waterfall_chart/waterfall/waterfall_flyout/waterfall_flyout_table.tsx

@@ -67,7 +67,7 @@ export const Table = (props: Props) => {
       </EuiText>
       <EuiSpacer size="s" />
       <TableWithoutHeader
-        tableLayout={'fixed'}
+        tableLayout="fixed"
         compressed
         responsiveBreakpoint={false}
         columns={columns}

x-pack/plugins/observability_solution/synthetics/public/apps/synthetics/components/test_now_mode/simple/ping_list/ping_list_table.tsx

@@ -127,7 +127,7 @@ export function PingListTable({ loading, error, pings, onChange }: Props) {
               defaultMessage: 'No history found',
             })
       }
-      tableLayout={'auto'}
+      tableLayout="auto"
       onChange={onChange}
     />
   );

x-pack/plugins/observability_solution/uptime/public/legacy_uptime/components/monitor/ping_list/ping_list_table.tsx

@@ -239,7 +239,7 @@ export function PingListTable({ loading, error, pings, pagination, onChange, fai
               defaultMessage: 'No history found',
             })
       }
-      tableLayout={'auto'}
+      tableLayout="auto"
       rowProps={getRowProps}
       onChange={onChange}
     />

x-pack/plugins/observability_solution/uptime/public/legacy_uptime/components/monitor/synthetics/waterfall/components/waterfall_flyout_table.tsx

@@ -67,7 +67,7 @@ export const Table = (props: Props) => {
       </EuiText>
       <EuiSpacer size="s" />
       <TableWithoutHeader
-        tableLayout={'fixed'}
+        tableLayout="fixed"
         compressed
         responsiveBreakpoint={false}
         columns={columns}

x-pack/plugins/observability_solution/uptime/public/legacy_uptime/components/overview/monitor_list/monitor_list.tsx

@@ -233,7 +233,7 @@ export const MonitorListComponent: ({
           <NoItemsMessage loading={Boolean(loading || isPending)} filters={filters} />
         }
         columns={columns}
-        tableLayout={'auto'}
+        tableLayout="auto"
         rowProps={
           hideExtraColumns
             ? ({ monitor_id: monitorId }) => ({

x-pack/plugins/observability_solution/uptime/public/legacy_uptime/components/synthetics/check_steps/steps_list.tsx

@@ -247,7 +247,7 @@ export const StepsList = ({
                 defaultMessage: 'No history found',
               })
         }
-        tableLayout={'auto'}
+        tableLayout="auto"
         rowProps={getRowProps}
       />
     </>

x-pack/plugins/security_solution/public/entity_analytics/components/entity_details_flyout/tabs/risk_inputs/risk_inputs_tab.tsx

@@ -186,7 +186,7 @@ export const RiskInputsTab = ({ entityType, entityName }: RiskInputsTabProps) =>
       <RiskInputsUtilityBar pagination={pagination} selectedAlerts={selectedItems} />
       <EuiSpacer size="xs" />
       <EuiInMemoryTable
-        compressed={true}
+        compressed
         loading={loadingRiskScore || loadingAlerts}
         items={alertsData ?? []}
         columns={alertsColumns}