[EuiSuperSelect] Remove popoverClassName and repositionOnScroll props

CHANGELOG.md

@@ -12,6 +12,10 @@
 
 - Deprecated `PartitionConfig` in favor of inclusion in Charts `theme.partition` ([#5492](https://github.com/elastic/eui/pull/5492))
 
+**Breaking changes**
+
+- Removed `popoverClassName` and `repositionOnScroll` props from `EuiSuperSelect` (use `popoverProps` instead) ([#5512](https://github.com/elastic/eui/pull/5512))
+
 ## [`43.1.1`](https://github.com/elastic/eui/tree/v43.1.1)
 
 **Bug fixes**

src-docs/src/views/flyout/flyout_complicated.js

@@ -232,7 +232,7 @@ export default () => {
                 valueOfSelected={superSelectvalue}
                 onChange={(value) => onSuperSelectChange(value)}
                 itemLayoutAlign="top"
-                repositionOnScroll={true}
+                popoverProps={{ repositionOnScroll: true }}
                 hasDividers
               />
             </EuiFormRow>

src/components/form/super_select/super_select.tsx

@@ -76,30 +76,14 @@ export type EuiSuperSelectProps<T extends string> = CommonProps &
 
     /**
      * Optional props to pass to the underlying [EuiPopover](/#/layout/popover).
-     *  Allows fine-grained control of the popover dropdown menu, including
+     * Allows fine-grained control of the popover dropdown menu, including
      * `repositionOnScroll` for EuiSuperSelects used within scrollable containers,
      * and customizing popover panel styling.
      *
      * Does not accept a nested `popoverProps.isOpen` property - use the top level
      * `isOpen` API instead.
      */
     popoverProps?: Partial<CommonProps & Omit<EuiPopoverProps, 'isOpen'>>;
-
-    /**
-     * Applied to the outermost wrapper (popover)
-     *
-     * **DEPRECATED: Use `popoverProps.className` instead (will take precedence over this prop if set).**
-     */
-    popoverClassName?: string;
-
-    /**
-     * When `true`, the popover's position is re-calculated when the user
-     * scrolls. When nesting an `EuiSuperSelect` in a scrollable container,
-     * `repositionOnScroll` should be `true`
-     *
-     * **DEPRECATED: Use `popoverProps.repositionOnScroll` instead (will take precedence over this prop if set).**
-     */
-    repositionOnScroll?: boolean;
   };
 
 export class EuiSuperSelect<T extends string> extends Component<
@@ -276,16 +260,14 @@ export class EuiSuperSelect<T extends string> extends Component<
       itemClassName,
       itemLayoutAlign,
       fullWidth,
-      popoverClassName,
       popoverProps,
       compressed,
-      repositionOnScroll,
       ...rest
     } = this.props;
 
     const popoverClasses = classNames(
       'euiSuperSelect',
-      popoverProps?.className ?? popoverClassName
+      popoverProps?.className
     );
 
     const buttonClasses = classNames(
@@ -346,7 +328,6 @@ export class EuiSuperSelect<T extends string> extends Component<
       <EuiInputPopover
         closePopover={this.closePopover}
         panelPaddingSize="none"
-        repositionOnScroll={repositionOnScroll}
         {...popoverProps}
         className={popoverClasses}
         isOpen={isOpen || this.state.isPopoverOpen}