@patternfly/react-core 4.222.2 → 4.222.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.222.2",
+  "version": "4.222.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",
@@ -34,16 +34,16 @@
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.73.2",
-    "@patternfly/react-styles": "^4.72.2",
-    "@patternfly/react-tokens": "^4.74.2",
+    "@patternfly/react-icons": "^4.73.5",
+    "@patternfly/react-styles": "^4.72.5",
+    "@patternfly/react-tokens": "^4.74.5",
     "focus-trap": "6.9.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
     "tslib": "^2.0.0"
   },
   "devDependencies": {
-    "@patternfly/patternfly": "4.196.7",
+    "@patternfly/patternfly": "4.196.8",
     "@rollup/plugin-commonjs": "^21.0.0",
     "@rollup/plugin-node-resolve": "^13.0.0",
     "@rollup/plugin-replace": "^3.0.0",
@@ -60,5 +60,5 @@
     "react": "^16.8.0 || ^17.0.0",
     "react-dom": "^16.8.0 || ^17.0.0"
   },
-  "gitHead": "7c58918e8fab9b30e51e24dce84141b9a7a063d2"
+  "gitHead": "5e748b4b49ef95edbf3bf15eb0684724f416f6c3"
 }

package/src/components/AboutModal/AboutModal.tsx

@@ -144,6 +144,6 @@ export class AboutModal extends React.Component<AboutModalProps, ModalState> {
         {...props}
       />,
       container
-    );
+    ) as React.ReactElement;
   }
 }

package/src/components/ApplicationLauncher/__tests__/Generated/__snapshots__/ApplicationLauncherSeparator.test.tsx.snap

@@ -3,7 +3,7 @@
 exports[`ApplicationLauncherSeparator should match snapshot 1`] = `
 <DocumentFragment>
   <li
-    role="separator"
+    role="none"
   >
     <div
       class="pf-c-divider"

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

@@ -42,7 +42,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
       role="menu"
     >
       <li
-        role="menuitem"
+        role="none"
       >
         <a
           aria-disabled="false"
@@ -50,13 +50,14 @@ exports[`ApplicationLauncher custom icon 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-7"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
         >
           Link
         </a>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <button
           aria-disabled="false"
@@ -64,6 +65,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-8"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
           type="button"
         >
@@ -71,7 +73,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
         </button>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <a
           aria-disabled="true"
@@ -79,13 +81,14 @@ exports[`ApplicationLauncher custom icon 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-9"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
         >
           Disabled Link
         </a>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <button
           aria-disabled="true"
@@ -93,6 +96,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-10"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
           type="button"
         >
@@ -100,7 +104,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
         </button>
       </li>
       <li
-        role="separator"
+        role="none"
       >
         <div
           class="pf-c-divider"
@@ -108,7 +112,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
         />
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <a
           aria-disabled="false"
@@ -116,13 +120,14 @@ exports[`ApplicationLauncher custom icon 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-11"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
         >
           Separated Link
         </a>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <button
           aria-disabled="false"
@@ -130,6 +135,7 @@ exports[`ApplicationLauncher custom icon 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-12"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
           type="button"
         >
@@ -260,7 +266,7 @@ exports[`ApplicationLauncher expanded 1`] = `
       role="menu"
     >
       <li
-        role="menuitem"
+        role="none"
       >
         <a
           aria-disabled="false"
@@ -268,13 +274,14 @@ exports[`ApplicationLauncher expanded 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-1"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
         >
           Link
         </a>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <button
           aria-disabled="false"
@@ -282,6 +289,7 @@ exports[`ApplicationLauncher expanded 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-2"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
           type="button"
         >
@@ -289,7 +297,7 @@ exports[`ApplicationLauncher expanded 1`] = `
         </button>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <a
           aria-disabled="true"
@@ -297,13 +305,14 @@ exports[`ApplicationLauncher expanded 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-3"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
         >
           Disabled Link
         </a>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <button
           aria-disabled="true"
@@ -311,6 +320,7 @@ exports[`ApplicationLauncher expanded 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-4"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
           type="button"
         >
@@ -318,7 +328,7 @@ exports[`ApplicationLauncher expanded 1`] = `
         </button>
       </li>
       <li
-        role="separator"
+        role="none"
       >
         <div
           class="pf-c-divider"
@@ -326,7 +336,7 @@ exports[`ApplicationLauncher expanded 1`] = `
         />
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <a
           aria-disabled="false"
@@ -334,13 +344,14 @@ exports[`ApplicationLauncher expanded 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-5"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
         >
           Separated Link
         </a>
       </li>
       <li
-        role="menuitem"
+        role="none"
       >
         <button
           aria-disabled="false"
@@ -348,6 +359,7 @@ exports[`ApplicationLauncher expanded 1`] = `
           data-ouia-component-id="OUIA-Generated-DropdownItem-6"
           data-ouia-component-type="PF4/DropdownItem"
           data-ouia-safe="true"
+          role="menuitem"
           tabindex="-1"
           type="button"
         >

package/src/components/Dropdown/InternalDropdownItem.tsx

@@ -211,7 +211,7 @@ export class InternalDropdownItem extends React.Component<InternalDropdownItemPr
         ...(styleChildren && {
           className: css(element.props.className, classes)
         }),
-        ...(this.props.role !== 'separator' && { ref: this.componentRef })
+        ...(this.props.role !== 'separator' && { role, ref: this.componentRef })
       });
 
     const renderDefaultComponent = (tag: string) => {
@@ -240,6 +240,7 @@ export class InternalDropdownItem extends React.Component<InternalDropdownItemPr
           ref={this.ref}
           className={classes}
           id={componentID}
+          role={role}
         >
           {componentContent}
         </Component>
@@ -268,7 +269,7 @@ export class InternalDropdownItem extends React.Component<InternalDropdownItemPr
           return (
             <li
               className={listItemClassName || null}
-              role={role}
+              role="none"
               onKeyDown={this.onKeyDown}
               onClick={(event: any) => {
                 if (!isDisabled && !isAriaDisabled) {

package/src/components/Dropdown/__tests__/Generated/__snapshots__/DropdownSeparator.test.tsx.snap

@@ -3,7 +3,7 @@
 exports[`DropdownSeparator should match snapshot 1`] = `
 <DocumentFragment>
   <li
-    role="separator"
+    role="none"
   >
     <div
       class="pf-c-divider ''"

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

@@ -52,6 +52,7 @@ exports[`Dropdown alignment breakpoints 1`] = `
       >
         <a
           class="pf-c-dropdown__menu-item"
+          role="none"
         >
           Link
         </a>
@@ -61,6 +62,7 @@ exports[`Dropdown alignment breakpoints 1`] = `
       >
         <button
           class="pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Action
@@ -72,6 +74,7 @@ exports[`Dropdown alignment breakpoints 1`] = `
         <a
           aria-disabled="true"
           class="pf-m-disabled pf-c-dropdown__menu-item"
+          role="none"
         >
           Disabled Link
         </a>
@@ -82,13 +85,14 @@ exports[`Dropdown alignment breakpoints 1`] = `
         <button
           aria-disabled="true"
           class="pf-m-disabled pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Disabled Action
         </button>
       </li>
       <li
-        role="separator"
+        role="none"
       >
         <div
           class="pf-c-divider"
@@ -100,6 +104,7 @@ exports[`Dropdown alignment breakpoints 1`] = `
       >
         <a
           class="pf-c-dropdown__menu-item"
+          role="none"
         >
           Separated Link
         </a>
@@ -109,6 +114,7 @@ exports[`Dropdown alignment breakpoints 1`] = `
       >
         <button
           class="pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Separated Action
@@ -313,6 +319,7 @@ exports[`Dropdown expanded 1`] = `
       >
         <a
           class="pf-c-dropdown__menu-item"
+          role="none"
         >
           Link
         </a>
@@ -322,6 +329,7 @@ exports[`Dropdown expanded 1`] = `
       >
         <button
           class="pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Action
@@ -333,6 +341,7 @@ exports[`Dropdown expanded 1`] = `
         <a
           aria-disabled="true"
           class="pf-m-disabled pf-c-dropdown__menu-item"
+          role="none"
         >
           Disabled Link
         </a>
@@ -343,13 +352,14 @@ exports[`Dropdown expanded 1`] = `
         <button
           aria-disabled="true"
           class="pf-m-disabled pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Disabled Action
         </button>
       </li>
       <li
-        role="separator"
+        role="none"
       >
         <div
           class="pf-c-divider"
@@ -361,6 +371,7 @@ exports[`Dropdown expanded 1`] = `
       >
         <a
           class="pf-c-dropdown__menu-item"
+          role="none"
         >
           Separated Link
         </a>
@@ -370,6 +381,7 @@ exports[`Dropdown expanded 1`] = `
       >
         <button
           class="pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Separated Action
@@ -665,6 +677,7 @@ exports[`KebabToggle expanded 1`] = `
       >
         <a
           class="pf-c-dropdown__menu-item"
+          role="none"
         >
           Link
         </a>
@@ -674,6 +687,7 @@ exports[`KebabToggle expanded 1`] = `
       >
         <button
           class="pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Action
@@ -685,6 +699,7 @@ exports[`KebabToggle expanded 1`] = `
         <a
           aria-disabled="true"
           class="pf-m-disabled pf-c-dropdown__menu-item"
+          role="none"
         >
           Disabled Link
         </a>
@@ -695,13 +710,14 @@ exports[`KebabToggle expanded 1`] = `
         <button
           aria-disabled="true"
           class="pf-m-disabled pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Disabled Action
         </button>
       </li>
       <li
-        role="separator"
+        role="none"
       >
         <div
           class="pf-c-divider"
@@ -713,6 +729,7 @@ exports[`KebabToggle expanded 1`] = `
       >
         <a
           class="pf-c-dropdown__menu-item"
+          role="none"
         >
           Separated Link
         </a>
@@ -722,6 +739,7 @@ exports[`KebabToggle expanded 1`] = `
       >
         <button
           class="pf-c-dropdown__menu-item"
+          role="none"
           type="button"
         >
           Separated Action

package/src/components/Dropdown/__tests__/__snapshots__/InternalDropdownItem.test.tsx.snap

@@ -7,6 +7,7 @@ exports[`InternalDropdownItem dropdown items a 1`] = `
   >
     <a
       class=""
+      role="none"
     >
       Something
     </a>
@@ -22,6 +23,7 @@ exports[`InternalDropdownItem dropdown items aria-disabled a 1`] = `
     <a
       aria-disabled="true"
       class="pf-m-aria-disabled"
+      role="none"
     >
       Something
     </a>
@@ -37,6 +39,7 @@ exports[`InternalDropdownItem dropdown items aria-disabled button 1`] = `
     <button
       aria-disabled="true"
       class="pf-m-aria-disabled"
+      role="none"
       type="button"
     >
       Something
@@ -52,6 +55,7 @@ exports[`InternalDropdownItem dropdown items button 1`] = `
   >
     <button
       class=""
+      role="none"
       type="button"
     >
       Something
@@ -67,6 +71,7 @@ exports[`InternalDropdownItem dropdown items description a 1`] = `
   >
     <a
       class="pf-m-description"
+      role="none"
     >
       <div
         class="pf-c-dropdown__menu-item-main"
@@ -90,6 +95,7 @@ exports[`InternalDropdownItem dropdown items description button 1`] = `
   >
     <button
       class="pf-m-description"
+      role="none"
       type="button"
     >
       <div
@@ -115,6 +121,7 @@ exports[`InternalDropdownItem dropdown items disabled a 1`] = `
     <a
       aria-disabled="true"
       class=""
+      role="none"
     >
       Something
     </a>
@@ -130,6 +137,7 @@ exports[`InternalDropdownItem dropdown items disabled button 1`] = `
     <button
       aria-disabled="true"
       class=""
+      role="none"
       type="button"
     >
       Something
@@ -145,6 +153,7 @@ exports[`InternalDropdownItem dropdown items hover a 1`] = `
   >
     <a
       class=""
+      role="none"
     >
       Something
     </a>
@@ -159,6 +168,7 @@ exports[`InternalDropdownItem dropdown items hover button 1`] = `
   >
     <button
       class=""
+      role="none"
       type="button"
     >
       Something

package/src/components/Dropdown/examples/DropdownBasic.tsx

@@ -19,8 +19,10 @@ export const DropdownBasic: React.FunctionComponent = () => {
   };
 
   const dropdownItems = [
-    <DropdownItem key="link">Link</DropdownItem>,
-    <DropdownItem key="action" component="button">
+    <DropdownItem key="link" tooltip="Tooltip for enabled link">
+      Link
+    </DropdownItem>,
+    <DropdownItem key="action" component="button" tooltip="Tooltip for enabled button">
       Action
     </DropdownItem>,
     <DropdownItem key="disabled link" isDisabled href="www.google.com">