@patternfly/react-core 4.240.0 → 4.241.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.240.0",
+  "version": "4.241.1",
   "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",
@@ -44,9 +44,9 @@
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.91.0",
-    "@patternfly/react-styles": "^4.90.0",
-    "@patternfly/react-tokens": "^4.92.0",
+    "@patternfly/react-icons": "^4.92.1",
+    "@patternfly/react-styles": "^4.91.1",
+    "@patternfly/react-tokens": "^4.93.1",
     "focus-trap": "6.9.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
@@ -67,8 +67,8 @@
     "typescript": "^4.7.4"
   },
   "peerDependencies": {
-    "react": "^16.8.0 || ^17.0.0",
-    "react-dom": "^16.8.0 || ^17.0.0"
+    "react": "^16.8 || ^17 || ^18",
+    "react-dom": "^16.8 || ^17 || ^18"
   },
-  "gitHead": "2703bc1465144000deac49dab1b9d779093205c3"
+  "gitHead": "447aaa1f6cad3927b263c7eb737f16a4c3dde438"
 }

package/src/components/AlertGroup/examples/AlertGroupSingularDynamic.tsx

@@ -24,15 +24,15 @@ export const AlertGroupSingularDynamic: React.FunctionComponent = () => {
   const getUniqueId = () => new Date().getTime();
 
   const addSuccessAlert = () => {
-    addAlert('Toast success alert', 'success', getUniqueId());
+    addAlert('Success alert', 'success', getUniqueId());
   };
 
   const addDangerAlert = () => {
-    addAlert('Toast danger alert', 'danger', getUniqueId());
+    addAlert('Danger alert', 'danger', getUniqueId());
   };
 
   const addInfoAlert = () => {
-    addAlert('Toast info alert', 'info', getUniqueId());
+    addAlert('Info alert', 'info', getUniqueId());
   };
 
   return (

package/src/components/AlertGroup/examples/AlertGroupSingularDynamicOverflow.tsx

@@ -38,15 +38,15 @@ export const AlertGroupSingularDynamicOverflow: React.FunctionComponent = () =>
   const getUniqueId = () => new Date().getTime();
 
   const addSuccessAlert = () => {
-    addAlert('Toast success alert', 'success', getUniqueId());
+    addAlert('Success alert', 'success', getUniqueId());
   };
 
   const addDangerAlert = () => {
-    addAlert('Toast danger alert', 'danger', getUniqueId());
+    addAlert('Danger alert', 'danger', getUniqueId());
   };
 
   const addInfoAlert = () => {
-    addAlert('Toast info alert', 'info', getUniqueId());
+    addAlert('Info alert', 'info', getUniqueId());
   };
 
   const onOverflowClick = () => {

package/src/components/OverflowMenu/OverflowMenu.tsx

@@ -40,13 +40,13 @@ export class OverflowMenu extends React.Component<OverflowMenuProps, OverflowMen
   componentDidMount() {
     this.handleResize();
     if (canUseDOM) {
-      window.addEventListener('resize', debounce(this.handleResize, 250));
+      window.addEventListener('resize', this.handleResizeWithDelay);
     }
   }
 
   componentWillUnmount() {
     if (canUseDOM) {
-      window.removeEventListener('resize', debounce(this.handleResize, 250));
+      window.removeEventListener('resize', this.handleResizeWithDelay);
     }
   }
 
@@ -62,6 +62,8 @@ export class OverflowMenu extends React.Component<OverflowMenuProps, OverflowMen
     this.setState({ isBelowBreakpoint });
   };
 
+  handleResizeWithDelay = debounce(this.handleResize, 250);
+
   render() {
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const { className, breakpoint, children, ...props } = this.props;

package/src/components/Tabs/Tabs.tsx

@@ -461,7 +461,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
               )}
             </GenerateId>
           )}
-          {!isOverflowHorizontal && (
+          {showScrollButtons && (
             <button
               type="button"
               className={css(styles.tabsScrollButton, isSecondary && buttonStyles.modifiers.secondary)}
@@ -477,7 +477,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
             {isOverflowHorizontal ? filteredChildrenWithoutOverflow : filteredChildren}
             {hasOverflowTab && <OverflowTab overflowingTabs={overflowingTabProps} {...overflowObjectProps} />}
           </ul>
-          {!isOverflowHorizontal && (
+          {showScrollButtons && (
             <button
               type="button"
               className={css(styles.tabsScrollButton, isSecondary && buttonStyles.modifiers.secondary)}

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

@@ -411,3 +411,22 @@ test('should render tabs with secondary border bottom when passed hasSecondaryBo
 
   expect(tabsContainer).toHaveClass('pf-m-border-bottom');
 });
+
+test('should not render scroll buttons when isVertical is true', () => {
+  render(
+    <Tabs isVertical>
+      <Tab id="tab1" eventKey={0} title={<TabTitleText>"Tab item 1"</TabTitleText>}>
+        Tab 1 section
+      </Tab>
+      <Tab id="tab2" eventKey={1} title={<TabTitleText>"Tab item 2"</TabTitleText>}>
+        Tab 2 section
+      </Tab>
+      <Tab id="tab3" eventKey={2} title={<TabTitleText>"Tab item 3"</TabTitleText>}>
+        Tab 3 section
+      </Tab>
+    </Tabs>
+  );
+
+  expect(screen.queryByLabelText('Scroll left')).not.toBeInTheDocument();
+  expect(screen.queryByLabelText('Scroll right')).not.toBeInTheDocument();
+})