chore(Popover): remove deprecated props

patternfly · Oct 11, 2022 · 6a457bb · 6a457bb
1 parent d74be70
commit 6a457bb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 24 deletions.
diff --git a/packages/react-core/src/components/Popover/Popover.tsx b/packages/react-core/src/components/Popover/Popover.tsx
@@ -16,7 +16,7 @@ import { ReactElement } from 'react';
 import { FocusTrap } from '../../helpers';
 import { Popper, getOpacityTransition } from '../../helpers/Popper/Popper';
 import { getUniqueId } from '../../helpers/util';
-import { Instance as TippyInstance, Props as TippyProps } from '../../helpers/Popper/DeprecatedTippyTypes';
+import { Instance as TippyInstance } from '../../helpers/Popper/DeprecatedTippyTypes';
 
 export enum PopoverPosition {
   auto = 'auto',
@@ -60,10 +60,6 @@ export interface PopoverProps {
    * bodyContent={hide => <Button onClick={() => hide()}>Close</Button>}
    */
   bodyContent: React.ReactNode | ((hide: () => void) => React.ReactNode);
-  /** @deprecated - no longer used. if you want to constrain the popper to a specific element
-   * use the appendTo prop instead.
-   */
-  boundary?: 'scrollParent' | 'window' | 'viewport' | HTMLElement;
   /**
    * The reference element to which the popover is relatively placed to. If you cannot wrap
    * the reference with the Popover, you can use the reference prop instead.
@@ -215,8 +211,6 @@ export interface PopoverProps {
   shouldOpen?: (showFunction?: () => void, event?: MouseEvent | KeyboardEvent) => void;
   /** Flag indicating whether the close button should be shown. */
   showClose?: boolean;
-  /** @deprecated - no longer used. */
-  tippyProps?: Partial<TippyProps>;
   /** Whether to trap focus in the popover. */
   withFocusTrap?: boolean;
   /** The z-index of the popover. */
@@ -265,21 +259,12 @@ export const Popover: React.FunctionComponent<PopoverProps> = ({
   animationDuration = 300,
   id,
   withFocusTrap: propWithFocusTrap,
-  boundary,
-  tippyProps,
   reference,
   hasNoPadding = false,
   hasAutoWidth = false,
   removeFindDomNode = false,
   ...rest
 }: PopoverProps) => {
-  if (process.env.NODE_ENV !== 'production') {
-    boundary !== undefined &&
-      console.warn(
-        'The Popover boundary prop has been deprecated. If you want to constrain the popper to a specific element use the appendTo prop instead.'
-      );
-    tippyProps !== undefined && console.warn('The Popover tippyProps prop has been deprecated and is no longer used.');
-  }
   // could make this a prop in the future (true | false | 'toggle')
   // const hideOnClick = true;
   const uniqueId = id || getUniqueId();

diff --git a/packages/react-integration/demo-app-ts/src/components/demos/PopoverDemo/PopoverDemo.tsx b/packages/react-integration/demo-app-ts/src/components/demos/PopoverDemo/PopoverDemo.tsx
@@ -1,6 +1,5 @@
 import { Popover } from '@patternfly/react-core';
 import React, { Component } from 'react';
-import { Instance as TippyInstance } from '@patternfly/react-core/dist/esm/helpers/Popper/DeprecatedTippyTypes';
 
 export class PopoverDemo extends Component {
   popoverRef: React.RefObject<HTMLButtonElement>;
@@ -30,13 +29,6 @@ export class PopoverDemo extends Component {
         >
           {this.myPopoverProps.children}
         </Popover>
-        <Popover
-          bodyContent="test deprecated props"
-          tippyProps={{ duration: 0, delay: 0 }}
-          shouldClose={(tip: TippyInstance) => !tip}
-        >
-          <button>Trigger</button>
-        </Popover>
         <div>
           <button id="popover-selector">Popover attached via selector ref</button>
           <Popover