@patternfly/react-core 4.235.2 → 4.235.5

package/package.json

@@ -1,16 +1,10 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.235.2",
+  "version": "4.235.5",
   "description": "This library provides a set of common React components for use with the PatternFly reference implementation.",
   "main": "dist/js/index.js",
   "module": "dist/esm/index.js",
   "types": "dist/esm/index.d.ts",
-  "exports": {
-    ".": "./dist/esm/index.js",
-    "./next": "./dist/esm/next/index.js",
-    "./deprecated": "./dist/esm/deprecated/index.js",
-    "./package.json": "./package.json"
-  },
   "typesVersions": {
     "*": {
       "next": [
@@ -45,14 +39,14 @@
   },
   "homepage": "https://github.com/patternfly/patternfly-react#readme",
   "scripts": {
-    "build:umd": "rollup -c && rollup -c --environment IS_PRODUCTION",
+    "build:umd": "rollup -c --environment IS_PRODUCTION",
     "clean": "rimraf dist",
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.86.2",
-    "@patternfly/react-styles": "^4.85.2",
-    "@patternfly/react-tokens": "^4.87.2",
+    "@patternfly/react-icons": "^4.86.5",
+    "@patternfly/react-styles": "^4.85.5",
+    "@patternfly/react-tokens": "^4.87.5",
     "focus-trap": "6.9.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
@@ -76,5 +70,5 @@
     "react": "^16.8.0 || ^17.0.0",
     "react-dom": "^16.8.0 || ^17.0.0"
   },
-  "gitHead": "481223940736dfb78100455556d8aa23738cec66"
+  "gitHead": "6ec6b7c439ec2475f03ddfdc9ad494396aecb6a6"
 }

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

@@ -81,7 +81,7 @@ export const DualListSelectorComposableDragDrop: React.FunctionComponent = () =>
     }
   };
 
-  const onDrop = (source: DraggableItemPosition, dest?: DraggableItemPosition) => {
+  const onDrop = (source: DraggableItemPosition, dest: DraggableItemPosition | undefined) => {
     if (dest) {
       const newList = [...chosenOptions];
       const [removed] = newList.splice(source.index, 1);

package/src/components/NumberInput/NumberInput.tsx

@@ -10,7 +10,7 @@ import { TextInput } from '../TextInput';
 
 export interface NumberInputProps extends React.HTMLProps<HTMLDivElement> {
   /** Value of the number input */
-  value?: number;
+  value?: number | null;
   /** Additional classes added to the number input */
   className?: string;
   /** Sets the width of the number input to a number of characters */
@@ -89,6 +89,7 @@ export const NumberInput: React.FunctionComponent<NumberInputProps> = ({
   plusBtnProps,
   ...props
 }: NumberInputProps) => {
+  value = value || 0;
   const numberInputUnit = <div className={css(styles.numberInputUnit)}>{unit}</div>;
   const keyDownHandler =
     inputProps && inputProps.onKeyDown ? inputProps.onKeyDown : defaultKeyDownHandler({ inputName, onMinus, onPlus });

package/src/components/NumberInput/__tests__/NumberInput.test.tsx

@@ -169,4 +169,22 @@ describe('numberInput', () => {
   
     expect(input).toHaveDisplayValue('47.01');
   });
+
+  test('renders 0 if no value passed', () => {
+    render(<NumberInput />);
+    const input = screen.getByRole('spinbutton');
+    expect(input).toHaveDisplayValue('0');
+  });
+
+  test('renders 0 if undefined value passed', () => {
+    render(<NumberInput value={undefined} />);
+    const input = screen.getByRole('spinbutton');
+    expect(input).toHaveDisplayValue('0');
+  });
+
+  test('renders 0 if null value passed', () => {
+    render(<NumberInput value={null} />);
+    const input = screen.getByRole('spinbutton');
+    expect(input).toHaveDisplayValue('0');
+  });
 });

package/src/demos/ComposableMenu/examples/ComposableActionsMenu.tsx

@@ -36,7 +36,7 @@ export const ComposableActionsMenu: React.FunctionComponent = () => {
     };
   }, [isOpen, menuRef]);
 
-  const onSelect = (event?: React.MouseEvent, itemId?: string | number) => {
+  const onSelect = (event: React.MouseEvent | undefined, itemId: string | number | undefined) => {
     if (typeof itemId === 'string' || typeof itemId === 'undefined') {
       return;
     }

package/src/demos/ComposableMenu/examples/ComposableFlyout.tsx

@@ -1,9 +1,10 @@
 import React from 'react';
 import { MenuToggle, Menu, MenuContent, MenuList, MenuItem, Popper } from '@patternfly/react-core';
 
-// eslint-disable-next-line no-console
-const onSelect = (event?: React.MouseEvent, itemId?: string | number) => console.log('selected', itemId);
-
+/* eslint-disable no-console */
+const onSelect = (event: React.MouseEvent | undefined, itemId: string | number | undefined) =>
+  console.log('selected', itemId);
+/* eslint-enable no-console */
 interface FlyoutMenuProps {
   children?: React.ReactElement;
   depth: number;

package/src/demos/ComposableMenu/examples/ComposableSimpleCheckboxSelect.tsx

@@ -50,7 +50,7 @@ export const ComposableSimpleCheckboxSelect: React.FunctionComponent = () => {
     setIsOpen(!isOpen);
   };
 
-  const onSelect = (event?: React.MouseEvent, itemId?: string | number) => {
+  const onSelect = (event: React.MouseEvent | undefined, itemId: string | number | undefined) => {
     if (typeof itemId === 'string' || typeof itemId === 'undefined') {
       return;
     }

package/src/demos/ComposableMenu/examples/ComposableSimpleSelect.tsx

@@ -59,7 +59,7 @@ export const ComposableSimpleSelect: React.FunctionComponent = () => {
     </MenuToggle>
   );
 
-  function onSelect(event?: React.MouseEvent, itemId?: string | number) {
+  function onSelect(event: React.MouseEvent | undefined, itemId: string | number | undefined) {
     if (typeof itemId === 'undefined') {
       return;
     }

package/src/demos/ComposableMenu/examples/ComposableTypeaheadSelect.tsx

@@ -55,7 +55,7 @@ export const ComposableTypeaheadSelect: React.FunctionComponent = () => {
 
   const focusOnInput = () => menuToggleRef.current?.querySelector('input')?.focus();
 
-  const onMenuSelect = (event?: React.MouseEvent, itemId?: string | number) => {
+  const onMenuSelect = (event: React.MouseEvent | undefined, itemId: string | number | undefined) => {
     if (itemId) {
       setInputValue(itemId.toString());
       setIsSelected(true);
@@ -124,7 +124,7 @@ export const ComposableTypeaheadSelect: React.FunctionComponent = () => {
   };
 
   // Close the menu when a click occurs outside of the menu toggle content
-  const onDocumentClick = (event?: MouseEvent) => {
+  const onDocumentClick = (event: MouseEvent | undefined) => {
     if (!menuToggleRef.current?.contains(event?.target as HTMLElement)) {
       setIsMenuOpen(false);
     }