Drop types

Author: rebeccaalpert

package-lock.json

@@ -46,8 +46,8 @@
     "@patternfly/documentation-framework": "^6.5.16",
     "@patternfly/react-code-editor": "^6.0.0",
     "@patternfly/patternfly": "^6.0.0",
-    "@types/react": "^19.1.0",
-    "@types/react-dom": "^19.1.2",
+    "@types/react": "^18.2.33",
+    "@types/react-dom": "^18.3.1",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
     "typescript": "^5.8.3"

packages/module/package.json

@@ -1,9 +1,9 @@
-import type { FunctionComponent, CSSProperties } from 'react';
+import type { HTMLAttributes, DetailedHTMLProps, FunctionComponent, CSSProperties } from 'react';
 import { Fragment } from 'react';
 import clsx from 'clsx';
 import { createUseStyles } from 'react-jss';
 
-export interface AnsibleProps extends React.DetailedHTMLProps<React.HTMLAttributes<HTMLElement>, HTMLElement> {
+export interface AnsibleProps extends DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement> {
   /** Supported/unsupported variant flag */
   isSupported?: boolean;
   /** Red Hat Ansible Automation Platform technology icon */

packages/module/src/Ansible/Ansible.tsx

@@ -1,4 +1,4 @@
-import type { FunctionComponent } from 'react';
+import type { ReactNode, MouseEvent, FunctionComponent } from 'react';
 import { useState } from 'react';
 import {
   Button,
@@ -20,7 +20,7 @@ export interface ColumnManagementModalColumn {
   /** Internal identifier of a column by which table displayed columns are filtered. */
   key: string;
   /** The actual display name of the column possibly with a tooltip or icon. */
-  title: React.ReactNode;
+  title: ReactNode,
   /** If user changes checkboxes, the component will send back column array with this property altered. */
   isShown?: boolean;
   /** Set to false if the column should be hidden initially */

packages/module/src/ColumnManagementModal/ColumnManagementModal.tsx

@@ -1,4 +1,4 @@
-import type { PropsWithChildren, FunctionComponent } from 'react';
+import type { ReactNode, PropsWithChildren, FunctionComponent } from 'react';
 import {
   Flex,
   FlexItem,
@@ -24,19 +24,19 @@ export interface PageHeaderLinkProps extends ButtonProps {
 
 export interface ContentHeaderProps {
   /** Title for content header */
-  title: React.ReactNode;
+  title: ReactNode;
   /** Optional subtitle for content header */
-  subtitle?: React.ReactNode;
+  subtitle?: ReactNode;
   /** Optional link below subtitle */
   linkProps?: PageHeaderLinkProps;
   /** Optional icon for content header (appears to the left of the content header's title with a divider) */
-  icon?: React.ReactNode;
+  icon?: ReactNode;
   /** Optional label for content header (appears to the right of the content header's title) */
-  label?: React.ReactNode;
+  label?: ReactNode;
   /** Breadcrumbs component */
-  breadcrumbs?: React.ReactNode;
+  breadcrumbs?: ReactNode;
   /** Menu that appears to the far right of the title */
-  actionMenu?: React.ReactNode;
+  actionMenu?: ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
 }

packages/module/src/ContentHeader/ContentHeader.tsx

@@ -1,4 +1,4 @@
-import type { FunctionComponent } from 'react';
+import type { ReactNode, FunctionComponent } from 'react';
 import { Component } from 'react';
 import { ExpandableSection, Title } from '@patternfly/react-core';
 import { createUseStyles } from 'react-jss';
@@ -15,19 +15,19 @@ const useStyles = createUseStyles({
 
 export interface ErrorBoundaryProps {
   /** The title text to display on the error page */
-  headerTitle?: React.ReactNode;
+  headerTitle?: ReactNode;
   /** Indicates if the error is silent */
   silent?: boolean;
   /** The title text to display with the error */
-  errorTitle?: React.ReactNode;
+  errorTitle?: ReactNode;
   /** The description text to display with the error */
-  errorDescription?: React.ReactNode;
+  errorDescription?: ReactNode;
   /** The default description text to display with the error if no errorDescription is provided */
-  defaultErrorDescription?: React.ReactNode;
+  defaultErrorDescription?: ReactNode;
   /** The text for the toggle link that users can select to view error details */
   errorToggleText?: string;
   /** The component that the error boundary component is wrapped around, which should be returned if there is no error  */
-  children?: React.ReactNode;
+  children?: ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
   /** The heading level to use on the header title, default is h1 */

packages/module/src/ErrorBoundary/ErrorBoundary.tsx

@@ -10,7 +10,7 @@ import {
   StackItem, 
 } from '@patternfly/react-core';
 import { createUseStyles } from 'react-jss'
-import type { FunctionComponent } from 'react';
+import type { ReactNode, FunctionComponent } from 'react';
 
 const useStyles = createUseStyles({
   errorDescription: {
@@ -21,13 +21,13 @@ const useStyles = createUseStyles({
 /** extends EmptyStateProps */
 export interface ErrorStateProps extends Omit<EmptyStateProps, 'children' | 'titleText' | 'status'> {
   /** Title of the error. */
-  titleText?: React.ReactNode;
+  titleText?: ReactNode;
   /** A description of the error, if no body text is provided then it will be set to the defaultBodyText. */
-  bodyText?: React.ReactNode;
+  bodyText?: ReactNode;
   /** A default description of the error used if no errorDescription is provided. */
-  defaultBodyText?: React.ReactNode;
+  defaultBodyText?: ReactNode;
   /** Custom footer content */
-  customFooter?: React.ReactNode;
+  customFooter?: ReactNode;
   /** ErrorState OUIA ID */
   ouiaId?: string | number;
   /** Status of the error message. */

packages/module/src/ErrorState/ErrorState.tsx

@@ -1,12 +1,12 @@
-import type { FunctionComponent } from 'react';
+import type { ReactNode, FunctionComponent } from 'react';
 import { Alert, AlertVariant, CodeBlock, CodeBlockCode, Flex, FlexItem, FlexProps } from '@patternfly/react-core';
 
 /** extends FlexProps */
 export interface LogSnippetProps extends FlexProps {
   /** Log snippet or code to be displayed */
-  logSnippet?: React.ReactNode;
+  logSnippet?: ReactNode;
   /** Message to appear above the log snippet */
-  message: string | React.ReactNode;
+  message: string | ReactNode;
   /** Log snippet alert variant */
   variant?: AlertVariant;
   /** Custom OUIA ID */

packages/module/src/LogSnippet/LogSnippet.tsx

@@ -1,27 +1,27 @@
-import type { FunctionComponent } from 'react';
+import type { ReactNode, FunctionComponent } from 'react';
 import { Button, EmptyState, EmptyStateBody, EmptyStateFooter, EmptyStateProps, EmptyStateStatus, EmptyStateVariant } from '@patternfly/react-core';
 import { HourglassHalfIcon } from '@patternfly/react-icons';
 
 /** extends EmptyStateProps */
 export interface MaintenanceProps extends Omit<EmptyStateProps, 'children' | 'title'> {
     /** The title for the maintenance message */
-    titleText?: React.ReactNode;
+    titleText?: ReactNode;
     /** Custom body text */
-    bodyText?: React.ReactNode;
+    bodyText?: ReactNode;
     /** A default bodyText used if no bodyText is provided */
-    defaultBodyText?: React.ReactNode;
+    defaultBodyText?: ReactNode;
     /** Start time in a specific time zone */
-    startTime?: React.ReactNode;
+    startTime?: ReactNode;
     /** End time in a specific time zone */
-    endTime?: React.ReactNode;
+    endTime?: ReactNode;
     /** Time zone specification */
-    timeZone?: React.ReactNode;
+    timeZone?: ReactNode;
     /** Information link */
     redirectLinkUrl?: string;
     /** Information link title */
-    redirectLinkText?: React.ReactNode;
+    redirectLinkText?: ReactNode;
     /** Custom footer content */
-    customFooter?: React.ReactNode;
+    customFooter?: ReactNode;
     /** Custom OUIA ID */
     ouiaId?: string | number;  }
 

packages/module/src/Maintenance/Maintenance.tsx

@@ -1,4 +1,4 @@
-import type { FunctionComponent } from 'react';
+import type { ReactNode, FunctionComponent } from 'react';
 import { Button, EmptyState, EmptyStateBody, EmptyStateFooter, EmptyStateProps, EmptyStateVariant } from '@patternfly/react-core';
 import NotFoundIcon from '../NotFoundIcon';
 
@@ -7,11 +7,11 @@ export interface MissingPageProps extends Omit<EmptyStateProps, 'children' | 'ti
   /** The URL that the home page link points to */
   toHomePageUrl?: string;
   /** The text label for the link that points back to the home page */
-  toHomePageText?: React.ReactNode;
+  toHomePageText?: ReactNode;
   /** The title for the invalid object message */
-  titleText?: React.ReactNode;
+  titleText?: ReactNode;
   /** The body text for the invalid object message */
-  bodyText?: React.ReactNode;
+  bodyText?: ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
 }