@patternfly/react-core 4.242.2 → 4.242.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.242.2",
+  "version": "4.242.4",
   "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,16 +44,16 @@
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.92.3",
-    "@patternfly/react-styles": "^4.91.3",
-    "@patternfly/react-tokens": "^4.93.3",
+    "@patternfly/react-icons": "^4.92.4",
+    "@patternfly/react-styles": "^4.91.4",
+    "@patternfly/react-tokens": "^4.93.4",
     "focus-trap": "6.9.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
     "tslib": "^2.0.0"
   },
   "devDependencies": {
-    "@patternfly/patternfly": "4.215.1",
+    "@patternfly/patternfly": "4.216.0",
     "@rollup/plugin-commonjs": "^21.0.0",
     "@rollup/plugin-node-resolve": "^13.0.0",
     "@rollup/plugin-replace": "^3.0.0",
@@ -70,5 +70,5 @@
     "react": "^16.8 || ^17 || ^18",
     "react-dom": "^16.8 || ^17 || ^18"
   },
-  "gitHead": "e35f87ea426d87511edcb2a98124e460fe80e9b7"
+  "gitHead": "9c5cb016c12b1b532b590285d6607b0c21778960"
 }

package/src/components/Tabs/Tabs.tsx

@@ -114,7 +114,14 @@ const variantStyle = {
 };
 
 interface TabsState {
+  /** Used to signal if the scroll buttons should be used  */
+  enableScrollButtons: boolean;
+  /** Used to control if the scroll buttons should be shown to the user via the pf-m-scrollable class */
   showScrollButtons: boolean;
+  /** Used to control if the scroll buttons should be rendered. Rendering must occur before the scroll buttons are
+   * shown and rendering must be stopped after they stop being shown to preserve CSS transitions.
+   */
+  renderScrollButtons: boolean;
   disableLeftScrollButton: boolean;
   disableRightScrollButton: boolean;
   shownKeys: (string | number)[];
@@ -127,10 +134,13 @@ interface TabsState {
 export class Tabs extends React.Component<TabsProps, TabsState> {
   static displayName = 'Tabs';
   tabList = React.createRef<HTMLUListElement>();
+  leftScrollButtonRef = React.createRef<HTMLButtonElement>();
   constructor(props: TabsProps) {
     super(props);
     this.state = {
+      enableScrollButtons: false,
       showScrollButtons: false,
+      renderScrollButtons: false,
       disableLeftScrollButton: true,
       disableRightScrollButton: true,
       shownKeys: this.props.defaultActiveKey !== undefined ? [this.props.defaultActiveKey] : [this.props.activeKey], // only for mountOnEnter case
@@ -219,7 +229,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
       const container = this.tabList.current;
       let disableLeftScrollButton = true;
       let disableRightScrollButton = true;
-      let showScrollButtons = false;
+      let enableScrollButtons = false;
       let overflowingTabCount = 0;
 
       if (container && !this.props.isVertical && !isOverflowHorizontal) {
@@ -229,7 +239,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
         // get last element and check if it is in view
         const overflowOnRight = !isElementInView(container, container.lastChild as HTMLElement, false);
 
-        showScrollButtons = overflowOnLeft || overflowOnRight;
+        enableScrollButtons = overflowOnLeft || overflowOnRight;
 
         disableLeftScrollButton = !overflowOnLeft;
         disableRightScrollButton = !overflowOnRight;
@@ -240,7 +250,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
       }
 
       this.setState({
-        showScrollButtons,
+        enableScrollButtons,
         disableLeftScrollButton,
         disableRightScrollButton,
         overflowingTabCount
@@ -287,6 +297,13 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
     }
   };
 
+  hideScrollButtons = () => {
+    const { enableScrollButtons, renderScrollButtons, showScrollButtons } = this.state;
+    if (!enableScrollButtons && !showScrollButtons && renderScrollButtons) {
+      this.setState({ renderScrollButtons: false });
+    }
+  };
+
   componentDidMount() {
     if (!this.props.isVertical) {
       if (canUseDOM) {
@@ -304,11 +321,12 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
       }
     }
     clearTimeout(this.scrollTimeout);
+    this.leftScrollButtonRef.current?.removeEventListener('transitionend', this.hideScrollButtons);
   }
 
-  componentDidUpdate(prevProps: TabsProps) {
+  componentDidUpdate(prevProps: TabsProps, prevState: TabsState) {
     const { activeKey, mountOnEnter, isOverflowHorizontal, children } = this.props;
-    const { shownKeys, overflowingTabCount } = this.state;
+    const { shownKeys, overflowingTabCount, enableScrollButtons } = this.state;
     if (prevProps.activeKey !== activeKey && mountOnEnter && shownKeys.indexOf(activeKey) < 0) {
       this.setState({
         shownKeys: shownKeys.concat(activeKey)
@@ -327,6 +345,16 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
     if (isOverflowHorizontal && currentOverflowingTabCount) {
       this.setState({ overflowingTabCount: currentOverflowingTabCount + overflowingTabCount });
     }
+
+    if (!prevState.enableScrollButtons && enableScrollButtons) {
+      this.setState({ renderScrollButtons: true });
+      setTimeout(() => {
+        this.leftScrollButtonRef.current?.addEventListener('transitionend', this.hideScrollButtons);
+        this.setState({ showScrollButtons: true });
+      }, 100);
+    } else if (prevState.enableScrollButtons && !enableScrollButtons) {
+      this.setState({ showScrollButtons: false });
+    }
   }
 
   render() {
@@ -367,6 +395,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
     } = this.props;
     const {
       showScrollButtons,
+      renderScrollButtons,
       disableLeftScrollButton,
       disableRightScrollButton,
       shownKeys,
@@ -421,7 +450,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
             isVertical && expandable && formatBreakpointMods(expandable, styles),
             isVertical && expandable && isExpandedLocal && styles.modifiers.expanded,
             isBox && styles.modifiers.box,
-            showScrollButtons && !isVertical && styles.modifiers.scrollable,
+            showScrollButtons && styles.modifiers.scrollable,
             usePageInsets && styles.modifiers.pageInsets,
             !hasBorderBottom && styles.modifiers.noBorderBottom,
             hasSecondaryBorderBottom && styles.modifiers.borderBottom,
@@ -461,7 +490,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
               )}
             </GenerateId>
           )}
-          {showScrollButtons && (
+          {renderScrollButtons && (
             <button
               type="button"
               className={css(styles.tabsScrollButton, isSecondary && buttonStyles.modifiers.secondary)}
@@ -469,6 +498,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
               onClick={this.scrollLeft}
               disabled={disableLeftScrollButton}
               aria-hidden={disableLeftScrollButton}
+              ref={this.leftScrollButtonRef}
             >
               <AngleLeftIcon />
             </button>
@@ -477,7 +507,7 @@ export class Tabs extends React.Component<TabsProps, TabsState> {
             {isOverflowHorizontal ? filteredChildrenWithoutOverflow : filteredChildren}
             {hasOverflowTab && <OverflowTab overflowingTabs={overflowingTabProps} {...overflowObjectProps} />}
           </ul>
-          {showScrollButtons && (
+          {renderScrollButtons && (
             <button
               type="button"
               className={css(styles.tabsScrollButton, isSecondary && buttonStyles.modifiers.secondary)}

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

@@ -412,6 +412,25 @@ test('should render tabs with secondary border bottom when passed hasSecondaryBo
   expect(tabsContainer).toHaveClass('pf-m-border-bottom');
 });
 
+test('should not render scroll buttons by default', () => {
+  render(
+    <Tabs>
+      <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();
+})
+
 test('should not render scroll buttons when isVertical is true', () => {
   render(
     <Tabs isVertical>