@patternfly/react-core 4.225.4 → 4.225.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.225.4",
+  "version": "4.225.7",
   "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",
@@ -34,9 +34,9 @@
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.76.4",
-    "@patternfly/react-styles": "^4.75.4",
-    "@patternfly/react-tokens": "^4.77.4",
+    "@patternfly/react-icons": "^4.76.7",
+    "@patternfly/react-styles": "^4.75.7",
+    "@patternfly/react-tokens": "^4.77.7",
     "focus-trap": "6.9.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
@@ -60,5 +60,5 @@
     "react": "^16.8.0 || ^17.0.0",
     "react-dom": "^16.8.0 || ^17.0.0"
   },
-  "gitHead": "b0a8f3bf5de76e77e098c65502f37cd11558dedc"
+  "gitHead": "f819a9821412c2387151f719a8a587ab5bd1b7b7"
 }

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

@@ -15,23 +15,19 @@ export const ButtonProgress: React.FunctionComponent = () => {
   const [isUploading, setIsUploading] = React.useState<boolean>(false);
 
   const primaryLoadingProps = {} as LoadingPropsType;
-  if (isPrimaryLoading) {
-    primaryLoadingProps.spinnerAriaValueText = 'Loading';
-    primaryLoadingProps.spinnerAriaLabelledBy = 'primary-loading-button';
-    primaryLoadingProps.isLoading = true;
-  }
+  primaryLoadingProps.spinnerAriaValueText = 'Loading';
+  primaryLoadingProps.spinnerAriaLabelledBy = 'primary-loading-button';
+  primaryLoadingProps.isLoading = isPrimaryLoading;
+
   const secondaryLoadingProps = {} as LoadingPropsType;
-  if (isSecondaryLoading) {
-    secondaryLoadingProps.spinnerAriaValueText = 'Loading';
-    secondaryLoadingProps.spinnerAriaLabel = 'Content being loaded';
-    secondaryLoadingProps.isLoading = true;
-  }
+  secondaryLoadingProps.spinnerAriaValueText = 'Loading';
+  secondaryLoadingProps.spinnerAriaLabel = 'Content being loaded';
+  secondaryLoadingProps.isLoading = isSecondaryLoading;
+
   const uploadingProps = {} as LoadingPropsType;
-  if (isUploading) {
-    uploadingProps.spinnerAriaValueText = 'Loading';
-    uploadingProps.isLoading = true;
-    uploadingProps.spinnerAriaLabel = 'Uploading data';
-  }
+  uploadingProps.spinnerAriaValueText = 'Loading';
+  uploadingProps.spinnerAriaLabel = 'Uploading data';
+  uploadingProps.isLoading = isUploading;
 
   return (
     <React.Fragment>
@@ -53,8 +49,6 @@ export const ButtonProgress: React.FunctionComponent = () => {
         icon={<UploadIcon />}
         {...uploadingProps}
       />
-      <br />
-      <br />
     </React.Fragment>
   );
 };

package/src/components/Dropdown/DropdownItem.tsx

@@ -27,7 +27,7 @@ export interface DropdownItemProps extends InternalDropdownItemProps, OUIAProps
   isAriaDisabled?: boolean;
   /** Render dropdown item as non-interactive item */
   isPlainText?: boolean;
-  /** Forces display of the hover state of the element */
+  /** @deprecated Forces display of the hover state of the element */
   isHovered?: boolean;
   /** Default hyperlink location */
   href?: string;
@@ -56,7 +56,6 @@ export const DropdownItem: React.FunctionComponent<DropdownItemProps> = ({
   isDisabled = false,
   isAriaDisabled = false,
   isPlainText = false,
-  isHovered = false,
   href,
   tooltip,
   tooltipProps = {},
@@ -88,7 +87,6 @@ export const DropdownItem: React.FunctionComponent<DropdownItemProps> = ({
           isDisabled={isDisabled}
           isAriaDisabled={isAriaDisabled}
           isPlainText={isPlainText}
-          isHovered={isHovered}
           href={href}
           tooltip={tooltip}
           tooltipProps={tooltipProps}

package/src/components/Dropdown/InternalDropdownItem.tsx

@@ -25,8 +25,6 @@ export interface InternalDropdownItemProps extends React.HTMLProps<HTMLAnchorEle
   isAriaDisabled?: boolean;
   /** Render dropdown item as a non-interactive item */
   isPlainText?: boolean;
-  /** Forces display of the hover state of the element */
-  isHovered?: boolean;
   /** Default hyperlink location */
   href?: string;
   /** Tooltip to display when hovered over the item */
@@ -67,7 +65,6 @@ export class InternalDropdownItem extends React.Component<InternalDropdownItemPr
 
   static defaultProps: InternalDropdownItemProps = {
     className: '',
-    isHovered: false,
     component: 'a',
     role: 'none',
     isDisabled: false,
@@ -163,7 +160,6 @@ export class InternalDropdownItem extends React.Component<InternalDropdownItemPr
     const {
       className,
       children,
-      isHovered,
       context,
       onClick,
       component,

package/src/components/Switch/Switch.tsx

@@ -91,10 +91,9 @@ export class Switch extends React.Component<SwitchProps & OUIAProps, { ouiaState
           className={css(styles.switchInput)}
           type="checkbox"
           onChange={event => onChange(event.target.checked, event)}
-          {...([true, false].includes(defaultChecked) && { defaultChecked })}
-          {...(![true, false].includes(defaultChecked) && { checked: isChecked })}
+          {...(defaultChecked !== undefined ? { defaultChecked } : { checked: isChecked })}
           disabled={isDisabled}
-          aria-labelledby={isAriaLabelledBy ? `${this.id}-on` : null}
+          aria-labelledby={!isAriaLabelledBy ? null : `${this.id}-${isChecked !== true ? 'off' : 'on'}`}
           {...props}
         />
         {label !== undefined ? (

package/src/components/Switch/__tests__/__snapshots__/Switch.test.tsx.snap

@@ -282,7 +282,7 @@ exports[`Switch switch with only label is not checked 1`] = `
   >
     <input
       aria-label=""
-      aria-labelledby="switch-is-not-checked-on"
+      aria-labelledby="switch-is-not-checked-off"
       class="pf-c-switch__input"
       id="switch-is-not-checked"
       type="checkbox"

package/src/components/Switch/examples/Switch.md

@@ -171,8 +171,16 @@ import React from 'react';
 import { Switch } from '@patternfly/react-core';
 
 <React.Fragment>
-  <Switch id="uncontrolled-switch-on" label="Message when on" labelOff="Message when off" defaultChecked={false} />
+  <Switch
+    id="uncontrolled-switch-no-label"
+    aria-label="An uncontrolled switch with no label"
+    defaultChecked={false}
+  />
   <br />
-  <Switch id="uncontrolled-no-label-switch-on" aria-label="Message when on" defaultChecked={true} />
+  <Switch
+    id="uncontrolled-switch-with-label"
+    label="An uncontrolled switch"
+    defaultChecked={false}
+  />
 </React.Fragment>;
 ```

package/src/components/TreeView/TreeView.tsx

@@ -58,7 +58,7 @@ export interface TreeViewProps {
   /** Callback for item selection */
   onSelect?: (event: React.MouseEvent, item: TreeViewDataItem, parentItem: TreeViewDataItem) => void;
   /** Callback for item checkbox selection */
-  onCheck?: (event: React.ChangeEvent, item: TreeViewDataItem, parentItem: TreeViewDataItem) => void;
+  onCheck?: (event: React.ChangeEvent<HTMLInputElement>, item: TreeViewDataItem, parentItem: TreeViewDataItem) => void;
   /** Active items of tree view */
   activeItems?: TreeViewDataItem[];
   /** Internal. Parent item of a TreeViewListItem */

package/src/components/TreeView/TreeViewListItem.tsx

@@ -26,7 +26,7 @@ export interface TreeViewListItemProps {
   /** Callback for item selection. Note: calling event.preventDefault() will prevent the node from toggling. */
   onSelect?: (event: React.MouseEvent, item: TreeViewDataItem, parent: TreeViewDataItem) => void;
   /** Callback for item checkbox selection */
-  onCheck?: (event: React.ChangeEvent, item: TreeViewDataItem, parent: TreeViewDataItem) => void;
+  onCheck?: (event: React.ChangeEvent<HTMLInputElement>, item: TreeViewDataItem, parent: TreeViewDataItem) => void;
   /** Flag indicating if a tree view item has a checkbox */
   hasCheck?: boolean;
   /** Additional properties of the tree view item checkbox */
@@ -115,8 +115,8 @@ const TreeViewListItemBase: React.FunctionComponent<TreeViewListItemProps> = ({
     <span className={css(styles.treeViewNodeCheck)}>
       <input
         type="checkbox"
-        onChange={(evt: React.ChangeEvent) => onCheck && onCheck(evt, itemData, parentItem)}
-        onClick={(evt: React.MouseEvent) => evt.stopPropagation()}
+        onChange={evt => onCheck && onCheck(evt, itemData, parentItem)}
+        onClick={evt => evt.stopPropagation()}
         ref={elem => elem && (elem.indeterminate = checkProps.checked === null)}
         {...checkProps}
         checked={checkProps.checked === null ? false : checkProps.checked}