@patternfly/react-core 6.0.0-alpha.90 → 6.0.0-alpha.92

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.0.0-alpha.89","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.0.0-alpha.91","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.0.0-alpha.89","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.0.0-alpha.91","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.0.0-alpha.89","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.0.0-alpha.91","private":true}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "6.0.0-alpha.90",
+  "version": "6.0.0-alpha.92",
   "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",
@@ -47,15 +47,15 @@
     "clean:exports": "node scripts/cleanDistExports.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^6.0.0-alpha.31",
-    "@patternfly/react-styles": "^6.0.0-alpha.30",
-    "@patternfly/react-tokens": "^6.0.0-alpha.30",
+    "@patternfly/react-icons": "^6.0.0-alpha.32",
+    "@patternfly/react-styles": "^6.0.0-alpha.31",
+    "@patternfly/react-tokens": "^6.0.0-alpha.31",
     "focus-trap": "7.5.4",
     "react-dropzone": "^14.2.3",
     "tslib": "^2.6.2"
   },
   "devDependencies": {
-    "@patternfly/patternfly": "6.0.0-alpha.190",
+    "@patternfly/patternfly": "6.0.0-alpha.201",
     "case-anything": "^2.1.13",
     "css": "^3.0.0",
     "fs-extra": "^11.2.0"
@@ -64,5 +64,5 @@
     "react": "^17 || ^18",
     "react-dom": "^17 || ^18"
   },
-  "gitHead": "390b5bdf9230e199a0559a69b9a8e33b70e5dc10"
+  "gitHead": "ec6e789678fbd6d3de0379652c3f5607e081c117"
 }

package/src/components/Label/Label.tsx

@@ -357,6 +357,7 @@ export const Label: React.FunctionComponent<LabelProps> = ({
         className
       )}
       onClick={isOverflowLabel || isAddLabel ? onLabelClick : undefined}
+      {...(LabelComponent === 'button' && { type: 'button' })}
     >
       {!isEditableActive && labelComponentChild}
       {!isEditableActive && onClose && closeButton}

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

@@ -2,6 +2,7 @@ import React from 'react';
 
 import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
+import '@testing-library/jest-dom';
 
 import { Label } from '../Label';
 
@@ -110,6 +111,12 @@ describe('Label', () => {
     expect(asFragment()).toMatchSnapshot();
   });
 
+  test('renders with variant overflow and type is set to button ', () => {
+    const { asFragment } = render(<Label variant="overflow">Something</Label>);
+    expect(screen.getByRole('button')).toHaveAttribute('type', 'button');
+    expect(asFragment()).toMatchSnapshot();
+  });
+
   test('a button is not rendered when onClick is not passed', () => {
     render(<Label>Click me</Label>);
 

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

@@ -599,3 +599,22 @@ exports[`Label renders with outline variant 1`] = `
   </span>
 </DocumentFragment>
 `;
+
+exports[`Label renders with variant overflow and type is set to button  1`] = `
+<DocumentFragment>
+  <button
+    class="pf-v6-c-label pf-m-overflow"
+    type="button"
+  >
+    <span
+      class="pf-v6-c-label__content"
+    >
+      <span
+        class="pf-v6-c-label__text"
+      >
+        Something
+      </span>
+    </span>
+  </button>
+</DocumentFragment>
+`;

package/src/components/Label/__tests__/__snapshots__/LabelGroup.test.tsx.snap

@@ -69,6 +69,7 @@ exports[`LabelGroup label group compact 1`] = `
         >
           <button
             class="pf-v6-c-label pf-m-overflow pf-m-compact"
+            type="button"
           >
             <span
               class="pf-v6-c-label__content"

package/src/components/MenuToggle/MenuToggle.tsx

@@ -201,13 +201,12 @@ class MenuToggleBase extends React.Component<MenuToggleProps, MenuToggleState> {
         >
           {splitButtonOptions?.items}
           <button
-            className={css(styles.menuToggleButton)}
+            className={css(styles.menuToggleButton, children && styles.modifiers.text)}
             type="button"
             aria-expanded={isExpanded}
             aria-label={ariaLabel}
             disabled={isDisabled}
             onClick={onClick}
-            {...(children && { style: { display: 'flex', paddingLeft: 'var(--pf-v5-global--spacer--sm)' } })}
             {...otherProps}
             {...ouiaProps}
           >