@patternfly/react-core 6.2.0-prerelease.3 → 6.2.0-prerelease.30

package/src/deprecated/components/Wizard/WizardToggle.tsx

@@ -8,7 +8,7 @@ import { WizardBody } from './WizardBody';
 
 export interface WizardToggleProps {
   /** Function that returns the WizardNav component */
-  nav: (isWizardNavOpen: boolean) => React.ReactElement;
+  nav: (isWizardNavOpen: boolean) => React.ReactElement<any>;
   /** The wizard steps */
   steps: WizardStep[];
   /** The currently active WizardStep */

package/src/deprecated/components/Wizard/__tests__/Generated/WizardToggle.test.tsx

@@ -10,7 +10,7 @@ import {} from '../..';
 it('WizardToggle should match snapshot (auto-generated)', () => {
   const { asFragment } = render(
     <WizardToggle
-      nav={(_isWizardNavOpen: boolean) => undefined as React.ReactElement}
+      nav={(_isWizardNavOpen: boolean) => undefined as unknown as React.ReactElement<any>}
       steps={[]}
       activeStep={{ name: 'some step' }}
       children={<div>ReactNode</div>}

package/src/deprecated/components/Wizard/__tests__/Generated/__snapshots__/WizardFooterInternal.test.tsx.snap

@@ -15,7 +15,6 @@ exports[`WizardFooterInternal should match snapshot (auto-generated) 1`] = `
           class="pf-v6-c-action-list__item"
         >
           <button
-            aria-disabled="false"
             class="pf-v6-c-button pf-m-secondary"
             data-ouia-component-id="OUIA-Generated-Button-secondary-1"
             data-ouia-component-type="PF6/Button"
@@ -36,7 +35,6 @@ exports[`WizardFooterInternal should match snapshot (auto-generated) 1`] = `
           class="pf-v6-c-action-list__item"
         >
           <button
-            aria-disabled="false"
             class="pf-v6-c-button pf-m-primary"
             data-ouia-component-id="OUIA-Generated-Button-primary-1"
             data-ouia-component-type="PF6/Button"
@@ -60,7 +58,6 @@ exports[`WizardFooterInternal should match snapshot (auto-generated) 1`] = `
           class="pf-v6-c-action-list__item"
         >
           <button
-            aria-disabled="false"
             class="pf-v6-c-button pf-m-link"
             data-ouia-component-id="OUIA-Generated-Button-link-1"
             data-ouia-component-type="PF6/Button"

package/src/deprecated/components/Wizard/__tests__/__snapshots__/Wizard.test.tsx.snap

@@ -12,7 +12,6 @@ exports[`Wizard Expandable Nav Wizard should match snapshot 1`] = `
         class="pf-v6-c-wizard__close"
       >
         <button
-          aria-disabled="false"
           aria-label="Close"
           class="pf-v6-c-button pf-m-plain"
           data-ouia-component-id="OUIA-Generated-Button-plain-2"
@@ -252,7 +251,6 @@ exports[`Wizard Expandable Nav Wizard should match snapshot 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-secondary"
                 data-ouia-component-id="OUIA-Generated-Button-secondary-2"
                 data-ouia-component-type="PF6/Button"
@@ -271,7 +269,6 @@ exports[`Wizard Expandable Nav Wizard should match snapshot 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-primary"
                 data-ouia-component-id="OUIA-Generated-Button-primary-2"
                 data-ouia-component-type="PF6/Button"
@@ -293,7 +290,6 @@ exports[`Wizard Expandable Nav Wizard should match snapshot 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-link"
                 data-ouia-component-id="OUIA-Generated-Button-link-2"
                 data-ouia-component-type="PF6/Button"
@@ -327,7 +323,6 @@ exports[`Wizard Wizard should match snapshot 1`] = `
         class="pf-v6-c-wizard__close"
       >
         <button
-          aria-disabled="false"
           aria-label="Close"
           class="pf-v6-c-button pf-m-plain"
           data-ouia-component-id="OUIA-Generated-Button-plain-1"
@@ -547,7 +542,6 @@ exports[`Wizard Wizard should match snapshot 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-secondary"
                 data-ouia-component-id="OUIA-Generated-Button-secondary-1"
                 data-ouia-component-type="PF6/Button"
@@ -566,7 +560,6 @@ exports[`Wizard Wizard should match snapshot 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-primary"
                 data-ouia-component-id="OUIA-Generated-Button-primary-1"
                 data-ouia-component-type="PF6/Button"
@@ -588,7 +581,6 @@ exports[`Wizard Wizard should match snapshot 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-link"
                 data-ouia-component-id="OUIA-Generated-Button-link-1"
                 data-ouia-component-type="PF6/Button"
@@ -710,7 +702,6 @@ exports[`Wizard bare wiz 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-secondary"
                 data-ouia-component-id="OUIA-Generated-Button-secondary-3"
                 data-ouia-component-type="PF6/Button"
@@ -729,7 +720,6 @@ exports[`Wizard bare wiz 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-primary"
                 data-ouia-component-id="OUIA-Generated-Button-primary-3"
                 data-ouia-component-type="PF6/Button"
@@ -751,7 +741,6 @@ exports[`Wizard bare wiz 1`] = `
               class="pf-v6-c-action-list__item"
             >
               <button
-                aria-disabled="false"
                 class="pf-v6-c-button pf-m-link"
                 data-ouia-component-id="OUIA-Generated-Button-link-3"
                 data-ouia-component-type="PF6/Button"

package/src/helpers/FocusTrap/FocusTrap.tsx

@@ -1,5 +1,5 @@
 import { createFocusTrap, FocusTrap as FocusTrapInstance, Options as FocusTrapOptions } from 'focus-trap';
-import React, { ComponentPropsWithRef, forwardRef, useEffect, useImperativeHandle, useRef } from 'react';
+import { ComponentPropsWithRef, forwardRef, useEffect, useImperativeHandle, useRef } from 'react';
 import { useUnmountEffect } from '../useUnmountEffect';
 
 export interface FocusTrapProps extends ComponentPropsWithRef<'div'> {

package/src/helpers/KeyboardHandler.tsx

@@ -182,7 +182,7 @@ export const setTabIndex = (options: HTMLElement[]) => {
  * @param event Event triggered by the keyboard
  * @param menuRef Menu reference
  */
-export const onToggleArrowKeydownDefault = (event: KeyboardEvent, menuRef: React.RefObject<HTMLDivElement>) => {
+export const onToggleArrowKeydownDefault = (event: KeyboardEvent, menuRef: React.RefObject<HTMLDivElement | null>) => {
   if (event.key !== 'ArrowDown' && event.key !== 'ArrowUp') {
     return;
   }

package/src/helpers/Popper/Popper.tsx

@@ -58,7 +58,7 @@ export interface PopperProps {
    */
   triggerRef?: HTMLElement | (() => HTMLElement) | React.RefObject<any>;
   /** The popper (menu/tooltip/popover) element */
-  popper: React.ReactElement;
+  popper: React.ReactElement<any>;
   /**
    * Reference to the popper (menu/tooltip/popover) element.
    * Passing this prop will remove the wrapper div element from the popper.
@@ -225,7 +225,7 @@ export const Popper: React.FunctionComponent<PopperProps> = ({
   const transitionTimerRef = React.useRef(null);
   const showTimerRef = React.useRef(null);
   const hideTimerRef = React.useRef(null);
-  const prevExitDelayRef = React.useRef<number>();
+  const prevExitDelayRef = React.useRef<number>(undefined);
 
   const refOrTrigger = refElement || triggerElement;
   const showPopper = isVisible || internalIsVisible;
@@ -527,7 +527,12 @@ export const Popper: React.FunctionComponent<PopperProps> = ({
     return popperRef ? (
       localPopper
     ) : (
-      <div style={{ display: 'contents' }} ref={(node) => setPopperElement(node?.firstElementChild as HTMLElement)}>
+      <div
+        style={{ display: 'contents' }}
+        ref={(node) => {
+          setPopperElement(node?.firstElementChild as HTMLElement);
+        }}
+      >
         {localPopper}
       </div>
     );
@@ -545,7 +550,12 @@ export const Popper: React.FunctionComponent<PopperProps> = ({
   return (
     <>
       {!triggerRef && trigger && React.isValidElement(trigger) && (
-        <div style={{ display: 'contents' }} ref={(node) => setTriggerElement(node?.firstElementChild as HTMLElement)}>
+        <div
+          style={{ display: 'contents' }}
+          ref={(node) => {
+            setTriggerElement(node?.firstElementChild as HTMLElement);
+          }}
+        >
           {trigger}
         </div>
       )}

package/src/helpers/Popper/__mocks__/Popper.tsx

@@ -1,4 +1,3 @@
-import React from 'react';
 import { PopperProps } from '../Popper';
 
 export const Popper = ({

package/src/helpers/Popper/thirdparty/react-popper/usePopper.ts

@@ -102,7 +102,7 @@ export const usePopper = (
     updateStateModifier
   ]);
 
-  const popperInstanceRef = React.useRef<any>();
+  const popperInstanceRef = React.useRef<any>(undefined);
 
   useIsomorphicLayoutEffect(() => {
     if (popperInstanceRef && popperInstanceRef.current) {

package/src/helpers/util.ts

@@ -1,4 +1,3 @@
-import * as ReactDOM from 'react-dom';
 import { globalWidthBreakpoints, globalHeightBreakpoints, SIDE } from './constants';
 
 /**
@@ -180,8 +179,8 @@ export function keyHandler(
     if (refsCollection[nextIndex].focus) {
       refsCollection[nextIndex].focus();
     }
-    // eslint-disable-next-line react/no-find-dom-node
-    const element = ReactDOM.findDOMNode(refsCollection[nextIndex]) as HTMLElement;
+
+    const element = refsCollection[nextIndex] as HTMLElement;
     element.focus();
   } else if (position !== 'tab') {
     if (isMultiDimensional) {

package/src/layouts/Bullseye/Bullseye.tsx

@@ -2,6 +2,8 @@ import * as React from 'react';
 import { css } from '@patternfly/react-styles';
 import styles from '@patternfly/react-styles/css/layouts/Bullseye/bullseye';
 
+import type { JSX } from 'react';
+
 export interface BullseyeProps extends React.HTMLProps<HTMLDivElement> {
   /** content rendered inside the Bullseye layout */
   children?: React.ReactNode;

package/src/layouts/Bullseye/__tests__/Bullseye.test.tsx

@@ -1,4 +1,4 @@
-import React from 'react';
+import React, { type JSX } from 'react';
 import { render, screen } from '@testing-library/react';
 import { Bullseye } from '../Bullseye';
 

package/src/components/Button/examples/ButtonRouterLink.tsx

@@ -1,9 +0,0 @@
-import React from 'react';
-import { Button } from '@patternfly/react-core';
-import { Link } from '@reach/router';
-
-export const ButtonRouterLink: React.FunctionComponent = () => (
-  <Button variant="link" component={(props: any) => <Link {...props} to="#" />}>
-    Router link
-  </Button>
-);

package/src/components/DataList/examples/DataListDraggable.tsx

@@ -1,108 +0,0 @@
-import React from 'react';
-import {
-  DataList,
-  DataListItem,
-  DataListCell,
-  DataListItemRow,
-  DataListCheck,
-  DataListControl,
-  DataListDragButton,
-  DataListItemCells,
-  getUniqueId
-} from '@patternfly/react-core';
-import { DragDrop, Draggable, Droppable } from '@patternfly/react-core/deprecated';
-
-interface ItemType {
-  id: string;
-  content: string;
-}
-
-const getItems = (count: number) =>
-  Array.from({ length: count }, (_, idx) => idx).map((idx) => ({
-    id: `draggable-item-${idx}`,
-    content: `item ${idx} `.repeat(idx === 4 ? 20 : 1)
-  }));
-
-const reorder = (list: ItemType[], startIndex: number, endIndex: number) => {
-  const result = list;
-  const [removed] = result.splice(startIndex, 1);
-  result.splice(endIndex, 0, removed);
-  return result;
-};
-
-export const DataListDraggable: React.FunctionComponent = () => {
-  const [items, setItems] = React.useState(getItems(10));
-  const [liveText, setLiveText] = React.useState('');
-
-  function onDrag(source) {
-    setLiveText(`Started dragging ${items[source.index].content}`);
-    // Return true to allow drag
-    return true;
-  }
-
-  function onDragMove(source, dest) {
-    const newText = dest ? `Move ${items[source.index].content} to ${items[dest.index].content}` : 'Invalid drop zone';
-    if (newText !== liveText) {
-      setLiveText(newText);
-    }
-  }
-
-  function onDrop(source, dest) {
-    if (dest) {
-      const newItems = reorder(items, source.index, dest.index);
-      setItems(newItems);
-
-      setLiveText('Dragging finished.');
-      return true; // Signal that this is a valid drop and not to animate the item returning home.
-    } else {
-      setLiveText('Dragging cancelled. List unchanged.');
-    }
-  }
-
-  const uniqueId = getUniqueId();
-
-  return (
-    <DragDrop onDrag={onDrag} onDragMove={onDragMove} onDrop={onDrop}>
-      <Droppable hasNoWrapper>
-        <DataList aria-label="draggable data list example" isCompact>
-          {items.map(({ id, content }) => (
-            <Draggable key={id} hasNoWrapper>
-              <DataListItem aria-labelledby={`draggable-${id}`} ref={React.createRef()}>
-                <DataListItemRow>
-                  <DataListControl>
-                    <DataListDragButton
-                      aria-label="Reorder"
-                      aria-labelledby={`draggable-${id}`}
-                      aria-describedby={`description-${uniqueId}`}
-                      aria-pressed="false"
-                    />
-                    <DataListCheck
-                      id={`check-draggable-${id}`}
-                      aria-labelledby={`draggable-${id}`}
-                      name={id}
-                      otherControls
-                    />
-                  </DataListControl>
-                  <DataListItemCells
-                    dataListCells={[
-                      <DataListCell key={id}>
-                        <span id={`draggable-${id}`}>{content}</span>
-                      </DataListCell>
-                    ]}
-                  />
-                </DataListItemRow>
-              </DataListItem>
-            </Draggable>
-          ))}
-        </DataList>
-      </Droppable>
-      <div className="pf-v6-screen-reader" aria-live="assertive">
-        {liveText}
-      </div>
-      <div className="pf-v6-screen-reader" id={`description-${uniqueId}`}>
-        Press space or enter to begin dragging, and use the arrow keys to navigate up or down. Press enter to confirm
-        the drag, or any other key to cancel the drag operation.
-      </div>
-    </DragDrop>
-  );
-};

package/src/components/DualListSelector/examples/DualListSelectorDragDrop.tsx

@@ -1,177 +0,0 @@
-import React from 'react';
-import {
-  DualListSelector,
-  DualListSelectorPane,
-  DualListSelectorList,
-  DualListSelectorListItem,
-  DualListSelectorControlsWrapper,
-  DualListSelectorControl
-} from '@patternfly/react-core';
-import { DragDrop, Draggable, Droppable, DraggableItemPosition } from '@patternfly/react-core/deprecated';
-import AngleDoubleLeftIcon from '@patternfly/react-icons/dist/esm/icons/angle-double-left-icon';
-import AngleLeftIcon from '@patternfly/react-icons/dist/esm/icons/angle-left-icon';
-import AngleDoubleRightIcon from '@patternfly/react-icons/dist/esm/icons/angle-double-right-icon';
-import AngleRightIcon from '@patternfly/react-icons/dist/esm/icons/angle-right-icon';
-
-export const DualListSelectorComposableDragDrop: React.FunctionComponent = () => {
-  const [ignoreNextOptionSelect, setIgnoreNextOptionSelect] = React.useState(false);
-  const [availableOptions, setAvailableOptions] = React.useState([
-    { text: 'Apple', selected: false, isVisible: true },
-    { text: 'Banana', selected: false, isVisible: true },
-    { text: 'Pineapple', selected: false, isVisible: true }
-  ]);
-  const [chosenOptions, setChosenOptions] = React.useState([
-    { text: 'Orange', selected: false, isVisible: true },
-    { text: 'Grape', selected: false, isVisible: true },
-    { text: 'Peach', selected: false, isVisible: true },
-    { text: 'Strawberry', selected: false, isVisible: true }
-  ]);
-
-  const moveSelected = (fromAvailable: boolean) => {
-    const sourceOptions = fromAvailable ? availableOptions : chosenOptions;
-    const destinationOptions = fromAvailable ? chosenOptions : availableOptions;
-    for (let i = 0; i < sourceOptions.length; i++) {
-      const option = sourceOptions[i];
-      if (option.selected && option.isVisible) {
-        sourceOptions.splice(i, 1);
-        destinationOptions.push(option);
-        option.selected = false;
-        i--;
-      }
-    }
-    if (fromAvailable) {
-      setAvailableOptions([...sourceOptions]);
-      setChosenOptions([...destinationOptions]);
-    } else {
-      setChosenOptions([...sourceOptions]);
-      setAvailableOptions([...destinationOptions]);
-    }
-  };
-
-  const moveAll = (fromAvailable: boolean) => {
-    if (fromAvailable) {
-      setChosenOptions([...availableOptions.filter((option) => option.isVisible), ...chosenOptions]);
-      setAvailableOptions([...availableOptions.filter((option) => !option.isVisible)]);
-    } else {
-      setAvailableOptions([...chosenOptions.filter((option) => option.isVisible), ...availableOptions]);
-      setChosenOptions([...chosenOptions.filter((option) => !option.isVisible)]);
-    }
-  };
-
-  const onOptionSelect = (
-    event: React.MouseEvent | React.ChangeEvent | React.KeyboardEvent,
-    index: number,
-    isChosen: boolean
-  ) => {
-    if (ignoreNextOptionSelect) {
-      setIgnoreNextOptionSelect(false);
-      return;
-    }
-    if (isChosen) {
-      const newChosen = [...chosenOptions];
-      newChosen[index].selected = !chosenOptions[index].selected;
-      setChosenOptions(newChosen);
-    } else {
-      const newAvailable = [...availableOptions];
-      newAvailable[index].selected = !availableOptions[index].selected;
-      setAvailableOptions(newAvailable);
-    }
-  };
-
-  const onDrop = (source: DraggableItemPosition, dest: DraggableItemPosition | undefined) => {
-    if (dest) {
-      const newList = [...chosenOptions];
-      const [removed] = newList.splice(source.index, 1);
-      newList.splice(dest.index, 0, removed);
-      setChosenOptions(newList);
-      return true;
-    }
-    return false;
-  };
-
-  return (
-    <DualListSelector>
-      <DualListSelectorPane
-        title="Available"
-        status={`${availableOptions.filter((option) => option.selected && option.isVisible).length} of ${
-          availableOptions.filter((option) => option.isVisible).length
-        } options selected`}
-      >
-        <DualListSelectorList>
-          {availableOptions.map((option, index) =>
-            option.isVisible ? (
-              <DualListSelectorListItem
-                key={index}
-                isSelected={option.selected}
-                id={`composable-drag-drop-available-option-${index}`}
-                onOptionSelect={(e) => onOptionSelect(e, index, false)}
-              >
-                {option.text}
-              </DualListSelectorListItem>
-            ) : null
-          )}
-        </DualListSelectorList>
-      </DualListSelectorPane>
-      <DualListSelectorControlsWrapper>
-        <DualListSelectorControl
-          isDisabled={!availableOptions.some((option) => option.selected)}
-          onClick={() => moveSelected(true)}
-          aria-label="Add selected"
-          icon={<AngleRightIcon />}
-        />
-        <DualListSelectorControl
-          isDisabled={availableOptions.length === 0}
-          onClick={() => moveAll(true)}
-          aria-label="Add all"
-          icon={<AngleDoubleRightIcon />}
-        />
-        <DualListSelectorControl
-          isDisabled={chosenOptions.length === 0}
-          onClick={() => moveAll(false)}
-          aria-label="Remove all"
-          icon={<AngleDoubleLeftIcon />}
-        />
-        <DualListSelectorControl
-          onClick={() => moveSelected(false)}
-          isDisabled={!chosenOptions.some((option) => option.selected)}
-          aria-label="Remove selected"
-          icon={<AngleLeftIcon />}
-        />
-      </DualListSelectorControlsWrapper>
-      <DragDrop
-        onDrag={() => {
-          setIgnoreNextOptionSelect(true);
-          return true;
-        }}
-        onDrop={onDrop}
-      >
-        <DualListSelectorPane
-          title="Chosen"
-          status={`${chosenOptions.filter((option) => option.selected && option.isVisible).length} of ${
-            chosenOptions.filter((option) => option.isVisible).length
-          } options selected`}
-          isChosen
-        >
-          <Droppable hasNoWrapper>
-            <DualListSelectorList>
-              {chosenOptions.map((option, index) =>
-                option.isVisible ? (
-                  <Draggable key={index} hasNoWrapper>
-                    <DualListSelectorListItem
-                      isSelected={option.selected}
-                      id={`composable-drag-drop-chosen-option-${index}`}
-                      onOptionSelect={(e) => onOptionSelect(e, index, true)}
-                      isDraggable
-                    >
-                      {option.text}
-                    </DualListSelectorListItem>
-                  </Draggable>
-                ) : null
-              )}
-            </DualListSelectorList>
-          </Droppable>
-        </DualListSelectorPane>
-      </DragDrop>
-    </DualListSelector>
-  );
-};

package/src/components/Toolbar/examples/ToolbarSpacers.tsx

@@ -1,48 +0,0 @@
-import React from 'react';
-import { Toolbar, ToolbarItem, ToolbarGroup, ToolbarContent } from '@patternfly/react-core';
-import { Button } from '@patternfly/react-core';
-
-export const ToolbarSpacers: React.FunctionComponent = () => {
-  const items = (
-    <React.Fragment>
-      <ToolbarGroup variant="action-group">
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-      </ToolbarGroup>
-      <ToolbarItem variant="separator"></ToolbarItem>
-      <ToolbarGroup variant="action-group" gap={{ lg: 'gapSm' }}>
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-        <ToolbarItem>
-          <Button variant="primary">Action</Button>
-        </ToolbarItem>
-      </ToolbarGroup>
-      <ToolbarItem variant="separator"></ToolbarItem>
-      <ToolbarGroup variant="action-group" gap={{ lg: 'gapLg' }}>
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-        <ToolbarItem>
-          <Button variant="secondary">Action</Button>
-        </ToolbarItem>
-      </ToolbarGroup>
-    </React.Fragment>
-  );
-
-  return (
-    <Toolbar id="toolbar-spacers">
-      <ToolbarContent>{items}</ToolbarContent>
-    </Toolbar>
-  );
-};