@patternfly/react-core 4.242.7 → 4.242.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.242.7",
+  "version": "4.242.8",
   "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",
@@ -70,5 +70,5 @@
     "react": "^16.8 || ^17 || ^18",
     "react-dom": "^16.8 || ^17 || ^18"
   },
-  "gitHead": "490a8d50cdf86999effdf59eca4ab3b7bf0c01eb"
+  "gitHead": "13e7bc690a3e25b2a6c14b2a4fef153fff2b70cb"
 }

package/src/components/Menu/Menu.tsx

@@ -101,7 +101,9 @@ class MenuBase extends React.Component<MenuProps, MenuState> {
     // Allow tabbing to first menu item
     const current = this.menuRef.current;
     if (current) {
-      const first = current.querySelector('ul button, ul a') as HTMLButtonElement | HTMLAnchorElement;
+      const first = current.querySelector('ul button:not(:disabled), ul a:not(:disabled)') as
+        | HTMLButtonElement
+        | HTMLAnchorElement;
       if (first) {
         first.tabIndex = 0;
       }

package/src/components/Select/Select.tsx

@@ -283,7 +283,13 @@ export class Select extends React.Component<SelectProps & OUIAProps, SelectState
 
     // Move focus to top of the menu if state.focusFirstOption was updated to true and the menu does not have custom content
     if (!prevState.focusFirstOption && this.state.focusFirstOption && !this.props.customContent) {
-      const firstRef = this.refCollection.find(ref => ref !== null);
+      const firstRef = this.refCollection.find(
+        ref =>
+          // If a select option is disabled then ref[0] will be undefined, so we want to return
+          // the first ref that both a) is not null and b) is not disabled.
+          ref !== null && ref[0]
+      );
+
       if (firstRef && firstRef[0]) {
         firstRef[0].focus();
       }

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

@@ -52,7 +52,7 @@ export const ComposableActionsMenu: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -47,7 +47,9 @@ export const ComposableApplicationLauncher: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button,input:not(:disabled)');
+        const firstElement = menuRef.current.querySelector(
+          'li > button:not(:disabled), li > a:not(:disabled), input:not(:disabled)'
+        );
         firstElement && (firstElement as HTMLElement).focus();
         setRefFullOptions(Array.from(menuRef.current.querySelectorAll('li:not(li[role=separator])')));
       }

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

@@ -103,7 +103,9 @@ export const ComposableContextSelector: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button,input:not(:disabled)');
+        const firstElement = menuRef.current.querySelector(
+          'li > button:not(:disabled), li > a:not(:disabled), input:not(:disabled)'
+        );
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -56,7 +56,7 @@ export const ComposableDrilldownMenu: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -58,7 +58,9 @@ export const ComposableDropdwnVariants: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button,input:not(:disabled),a');
+        const firstElement = menuRef.current.querySelector(
+          'li > button:not(:disabled), li > a:not(:disabled), input:not(:disabled)'
+        );
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -72,7 +72,7 @@ export const ComposableFlyout: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -37,7 +37,9 @@ export const ComposableOptionsMenuVariants: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button,input:not(:disabled)');
+        const firstElement = menuRef.current.querySelector(
+          'li > button:not(:disabled), li > a:not(:disabled), input:not(:disabled)'
+        );
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -43,7 +43,7 @@ export const ComposableSimpleCheckboxSelect: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li input:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -38,7 +38,7 @@ export const ComposableSimpleDropdown: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -37,7 +37,7 @@ export const ComposableSimpleSelect: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li > button:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);

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

@@ -210,7 +210,7 @@ export const ComposableTreeViewMenu: React.FunctionComponent = () => {
     ev.stopPropagation(); // Stop handleClickOutside from handling
     setTimeout(() => {
       if (menuRef.current) {
-        const firstElement = menuRef.current.querySelector('li button:not(:disabled)');
+        const firstElement = menuRef.current.querySelector('li > button:not(:disabled), li > a:not(:disabled)');
         firstElement && (firstElement as HTMLElement).focus();
       }
     }, 0);