@patternfly/react-core 6.3.1-prerelease.0 → 6.3.1-prerelease.1

package/helpers/package.json

@@ -1 +1 @@
-{"name":"@patternfly/react-core-helpers","main":"../dist/js/helpers/index.js","module":"../dist/esm/helpers/index.js","typings":"../dist/esm/helpers/index.d.ts","version":"6.3.0","private":true}
+{"name":"@patternfly/react-core-helpers","main":"../dist/js/helpers/index.js","module":"../dist/esm/helpers/index.js","typings":"../dist/esm/helpers/index.d.ts","version":"6.3.1-prerelease.0","private":true}

package/layouts/package.json

@@ -1 +1 @@
-{"name":"@patternfly/react-core-layouts","main":"../dist/js/layouts/index.js","module":"../dist/esm/layouts/index.js","typings":"../dist/esm/layouts/index.d.ts","version":"6.3.0","private":true}
+{"name":"@patternfly/react-core-layouts","main":"../dist/js/layouts/index.js","module":"../dist/esm/layouts/index.js","typings":"../dist/esm/layouts/index.d.ts","version":"6.3.1-prerelease.0","private":true}

package/next/package.json

@@ -1 +1 @@
-{"name":"@patternfly/react-core-next","main":"../dist/js/next/index.js","module":"../dist/esm/next/index.js","typings":"../dist/esm/next/index.d.ts","version":"6.3.0","private":true}
+{"name":"@patternfly/react-core-next","main":"../dist/js/next/index.js","module":"../dist/esm/next/index.js","typings":"../dist/esm/next/index.d.ts","version":"6.3.1-prerelease.0","private":true}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "6.3.1-prerelease.0",
+  "version": "6.3.1-prerelease.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",
@@ -63,5 +63,5 @@
     "react": "^17 || ^18 || ^19",
     "react-dom": "^17 || ^18 || ^19"
   },
-  "gitHead": "999dd945fd4f8ec281d9015586ac773833f16217"
+  "gitHead": "bc7a24267c0b8aff6f4ce5fd085aa6f01d5878bb"
 }

package/src/components/Page/PageToggleButton.tsx

@@ -14,6 +14,8 @@ export interface PageToggleButtonProps extends ButtonProps {
   'aria-label'?: string;
   /** Flag indicating whether the hamburger button variation with animations should be used. */
   isHamburgerButton?: boolean;
+  /** IsHamburgerButton must be true for hamburgerVariant to be have an effect. Adjusts and animates the hamburger icon to indicate what will happen upon clicking the button. */
+  hamburgerVariant?: 'expand' | 'collapse';
 }
 
 export const PageToggleButton: React.FunctionComponent<PageToggleButtonProps> = ({
@@ -23,6 +25,7 @@ export const PageToggleButton: React.FunctionComponent<PageToggleButtonProps> =
   id = 'nav-toggle',
   'aria-label': ariaLabel = 'Side navigation toggle',
   isHamburgerButton,
+  hamburgerVariant,
   ...props
 }: PageToggleButtonProps) => (
   <PageContextConsumer>
@@ -42,6 +45,10 @@ export const PageToggleButton: React.FunctionComponent<PageToggleButtonProps> =
           aria-expanded={sidebarOpen ? 'true' : 'false'}
           variant={ButtonVariant.plain}
           isHamburger={isHamburgerButton}
+          hamburgerVariant={hamburgerVariant}
+          {...(isHamburgerButton && {
+            isExpanded: sidebarOpen
+          })}
           {...props}
         >
           {!isHamburgerButton && children}

package/src/components/Page/__tests__/PageToggleButton.test.tsx

@@ -0,0 +1,74 @@
+import { render, screen } from '@testing-library/react';
+import { PageToggleButton } from '../PageToggleButton';
+import { PageContextProvider } from '../PageContext';
+
+test('Renders without children', () => {
+  render(
+    <div data-testid="container">
+      <PageToggleButton />
+    </div>
+  );
+
+  expect(screen.getByTestId('container').firstChild).toBeVisible();
+});
+
+test('Renders with children', () => {
+  render(<PageToggleButton>Test</PageToggleButton>);
+
+  expect(screen.getByText('Test')).toBeVisible();
+});
+
+test('Throws console error when isHamburger is true and isSidebarOpen is not passed', () => {
+  const consoleError = jest.spyOn(console, 'error').mockImplementation();
+
+  render(<PageToggleButton isHamburger aria-label="Test" />);
+
+  expect(consoleError).toHaveBeenCalledWith(
+    'Button: when the isHamburger property is passed in, you must also pass in a boolean value to the isExpanded property. It is expected that a hamburger button controls the expansion of other content.'
+  );
+});
+
+test('Does not throw console error when isHamburger is true and isSidebarOpen is false', () => {
+  const consoleError = jest.spyOn(console, 'error').mockImplementation();
+
+  render(<PageToggleButton isSidebarOpen={false} isHamburger aria-label="Test" />);
+
+  expect(consoleError).not.toHaveBeenCalledWith(
+    'Button: when the isHamburger property is passed in, you must also pass in a boolean value to the isExpanded property. It is expected that a hamburger button controls the expansion of other content..'
+  );
+});
+
+test('Does not throw console error when isHamburger is true and isSidebarOpen is true', () => {
+  const consoleError = jest.spyOn(console, 'error').mockImplementation();
+
+  render(<PageToggleButton isSidebarOpen isHamburger aria-label="Test" />);
+
+  expect(consoleError).not.toHaveBeenCalledWith(
+    'Button: when the isHamburger property is passed in, you must also pass in a boolean value to the isExpanded property. It is expected that a hamburger button controls the expansion of other content..'
+  );
+});
+
+// assisted by AI/Cursor
+test('Does not throw console error when isHamburger is true, isSidebarOpen is not passed, but managedIsSidebarOpen is true', () => {
+  const consoleError = jest.spyOn(console, 'error').mockImplementation();
+
+  const mockPageContext = {
+    isManagedSidebar: true,
+    onSidebarToggle: () => null,
+    isSidebarOpen: true, // managedIsSidebarOpen is true
+    width: 1024,
+    height: 768,
+    getBreakpoint: () => 'lg' as const,
+    getVerticalBreakpoint: () => 'lg' as const
+  };
+
+  render(
+    <PageContextProvider value={mockPageContext}>
+      <PageToggleButton isHamburger aria-label="Test" />
+    </PageContextProvider>
+  );
+
+  expect(consoleError).not.toHaveBeenCalledWith(
+    'Button: when the isHamburger property is passed in, you must also pass in a boolean value to the isExpanded property. It is expected that a hamburger button controls the expansion of other content..'
+  );
+});