fix console errors in inspector (#156894)

Steps to view problem
* install sample data set
* Open lens visualization
* Open inspector. Notice console errors
<img width="300" alt="Screen Shot 2023-05-05 at 11 03 25 AM"
src="https://user-images.githubusercontent.com/373691/236521366-d8fb9302-e93b-4047-a0bf-d7c09dcc3ffb.png">

https://github.com/elastic/eui/pull/6566 removed `closeButtonAriaLabel`
prop from [EuiFlyout](https://elastic.github.io/eui/#/layout/flyout) EUI
75.0.0 (Effecting 8.8 and 8.9). FlyoutService spreads options into
`EuiFlyout`, resulting in `closeButtonAriaLabel` getting added to dom
and causing error.

`OverlayFlyoutOpenOptions` type added by
https://github.com/elastic/kibana/issues/37894. I replaced
`OverlayFlyoutOpenOptions` with `EuiFlyoutProps` to make it more clear
what props are accepted and provide stronger typing that stays in sync
with EUI typings

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>

packages/core/overlays/core-overlays-browser/src/flyout.ts

@@ -5,7 +5,7 @@
  * in compliance with, at your election, the Elastic License 2.0 or the Server
  * Side Public License, v 1.
  */
-import type { EuiFlyoutSize, EuiOverlayMaskProps } from '@elastic/eui';
+import type { EuiFlyoutProps } from '@elastic/eui';
 import type { MountPoint, OverlayRef } from '@kbn/core-mount-utils-browser';
 
 /**
@@ -28,20 +28,10 @@ export interface OverlayFlyoutStart {
 /**
  * @public
  */
-export interface OverlayFlyoutOpenOptions {
-  className?: string;
-  closeButtonAriaLabel?: string;
-  ownFocus?: boolean;
-  'data-test-subj'?: string;
-  'aria-label'?: string;
-  size?: EuiFlyoutSize;
-  maxWidth?: boolean | number | string;
-  hideCloseButton?: boolean;
-  outsideClickCloses?: boolean;
-  maskProps?: EuiOverlayMaskProps;
+export type OverlayFlyoutOpenOptions = Omit<EuiFlyoutProps, 'onClose'> & {
   /**
    * EuiFlyout onClose handler.
    * If provided the consumer is responsible for calling flyout.close() to close the flyout;
    */
   onClose?: (flyout: OverlayRef) => void;
-}
+};

src/plugins/inspector/public/plugin.tsx

@@ -112,7 +112,7 @@ export class InspectorPublicPlugin implements Plugin<Setup, Start> {
         ),
         {
           'data-test-subj': 'inspectorPanel',
-          closeButtonAriaLabel: closeButtonLabel,
+          closeButtonProps: { 'aria-label': closeButtonLabel },
         }
       );
     };

src/plugins/kibana_react/public/overlays/create_react_overlays.test.tsx

@@ -69,12 +69,12 @@ test('passes through flyout options when opening flyout', () => {
 
   overlays.openFlyout(<>foo</>, {
     'data-test-subj': 'foo',
-    closeButtonAriaLabel: 'bar',
+    closeButtonProps: { 'aria-label': 'bar' },
   });
 
   expect(coreOverlays.openFlyout.mock.calls[0][1]).toEqual({
     'data-test-subj': 'foo',
-    closeButtonAriaLabel: 'bar',
+    closeButtonProps: { 'aria-label': 'bar' },
   });
 });
 

x-pack/plugins/aiops/public/components/log_categorization/show_flyout.tsx

@@ -85,7 +85,7 @@ export async function showCategorizeFlyout(
         {
           'data-test-subj': 'aiopsCategorizeFlyout',
           ownFocus: true,
-          closeButtonAriaLabel: 'aiopsCategorizeFlyout',
+          closeButtonProps: { 'aria-label': 'aiopsCategorizeFlyout' },
           onClose: onFlyoutClose,
           size: 'l',
         }

x-pack/plugins/graph/public/components/workspace_layout/workspace_top_nav_menu.tsx

@@ -171,9 +171,11 @@ export const WorkspaceTopNavMenu = (props: WorkspaceTopNavMenuProps) => {
         ),
         {
           size: 'm',
-          closeButtonAriaLabel: i18n.translate('xpack.graph.settings.closeLabel', {
-            defaultMessage: 'Close',
-          }),
+          closeButtonProps: {
+            'aria-label': i18n.translate('xpack.graph.settings.closeLabel', {
+              defaultMessage: 'Close',
+            }),
+          },
           'data-test-subj': 'graphSettingsFlyout',
           ownFocus: true,
           className: 'gphSettingsFlyout',

x-pack/plugins/ml/public/embeddables/common/resolve_job_selection.tsx

@@ -92,7 +92,7 @@ export async function resolveJobSelection(
         {
           'data-test-subj': 'mlFlyoutJobSelector',
           ownFocus: true,
-          closeButtonAriaLabel: 'jobSelectorFlyout',
+          closeButtonProps: { 'aria-label': 'jobSelectorFlyout' },
         }
       );
 

x-pack/plugins/ml/public/embeddables/job_creation/common/create_flyout.tsx

@@ -71,7 +71,6 @@ export function createFlyout(
           'data-test-subj': 'mlFlyoutLayerSelector',
           ownFocus: true,
           onClose: onFlyoutClose,
-          // @ts-expect-error should take any number/string compatible with the CSS width attribute
           size: '35vw',
         }
       );