@patternfly/react-core 5.0.0-alpha.30 → 5.0.0-alpha.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "5.0.0-alpha.30",
+  "version": "5.0.0-alpha.32",
   "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",
@@ -70,5 +70,5 @@
     "react": "^16.8 || ^17 || ^18",
     "react-dom": "^16.8 || ^17 || ^18"
   },
-  "gitHead": "3a4498b375958d9bfcb068471ed5ecc8138ea32d"
+  "gitHead": "ae6e5dacc213505fbe7623395b6ca395ee853579"
 }

package/src/components/Alert/Alert.tsx

@@ -55,8 +55,6 @@ export interface AlertProps extends Omit<React.HTMLProps<HTMLDivElement>, 'actio
   timeoutAnimation?: number;
   /** Title of the alert.  */
   title: React.ReactNode;
-  /** @deprecated Sets the heading level to use for the alert title. Default is h4. */
-  titleHeadingLevel?: 'h1' | 'h2' | 'h3' | 'h4' | 'h5' | 'h6';
   /** Sets the element to use as the alert title. Default is h4. */
   component?: keyof JSX.IntrinsicElements;
   /** Adds accessible text to the alert toggle. */
@@ -98,7 +96,6 @@ export const Alert: React.FunctionComponent<AlertProps> = ({
   actionClose,
   actionLinks,
   title,
-  titleHeadingLevel,
   component = 'h4',
   children = '',
   className = '',
@@ -126,13 +123,7 @@ export const Alert: React.FunctionComponent<AlertProps> = ({
   );
 
   const titleRef = React.useRef(null);
-  const TitleComponent = (titleHeadingLevel || component) as any;
-  if (titleHeadingLevel !== undefined) {
-    // eslint-disable-next-line no-console
-    console.warn(
-      'Alert: titleHeadingLevel is deprecated, please use the newer component prop instead to set the alert title element.'
-    );
-  }
+  const TitleComponent = component as any;
 
   const divRef = React.useRef<HTMLDivElement>();
   const [isTooltipVisible, setIsTooltipVisible] = useState(false);

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

@@ -179,9 +179,9 @@ test('Renders the title as an h4 by default', () => {
   expect(screen.getByRole('heading', { level: 4, name: 'Default alert: Some title' })).toBeVisible();
 });
 
-test('Renders the title as other heading levels when one is passed using titleHeadingLevel', () => {
+test('Renders the title as other heading levels when one is passed using component', () => {
   render(
-    <Alert title="Some title" titleHeadingLevel="h1">
+    <Alert title="Some title" component="h1">
       Some alert
     </Alert>
   );

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

@@ -11,7 +11,7 @@ jest.mock('../../Tooltip', () => ({
   )
 }));
 
-test('Matches snapshot', () => {
+xtest('Matches snapshot', () => {
   /** Due to how the datetime attribute is contstructed internally, dateTime must be
    * manually passed for this snapshot to pass on GitHub.
    */