Add Katie's changes from other PR

rebeccaalpert · rebeccaalpert · commit 38405154f272 · 2025-05-07T10:34:45.000-04:00
diff --git a/packages/module/src/Ansible/Ansible.tsx b/packages/module/src/Ansible/Ansible.tsx
@@ -1,9 +1,9 @@
-import type { HTMLAttributes, DetailedHTMLProps, FunctionComponent, CSSProperties } from 'react';
+import type { FunctionComponent, CSSProperties } from 'react';
 import { Fragment } from 'react';
 import clsx from 'clsx';
 import { createUseStyles } from 'react-jss';
 
-export interface AnsibleProps extends DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement> {
+export interface AnsibleProps extends React.DetailedHTMLProps<React.HTMLAttributes<HTMLElement>, HTMLElement> {
   /** Supported/unsupported variant flag */
   isSupported?: boolean;
   /** Red Hat Ansible Automation Platform technology icon */
diff --git a/packages/module/src/ColumnManagementModal/ColumnManagementModal.tsx b/packages/module/src/ColumnManagementModal/ColumnManagementModal.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, MouseEvent, FunctionComponent } from 'react';
+import type { FunctionComponent } from 'react';
 import { useState } from 'react';
 import {
   Button,
@@ -18,33 +18,33 @@ import { ModalProps, Modal, ModalVariant } from '@patternfly/react-core/deprecat
 
 export interface ColumnManagementModalColumn {
   /** Internal identifier of a column by which table displayed columns are filtered. */
-  key: string,
+  key: string;
   /** The actual display name of the column possibly with a tooltip or icon. */
-  title: ReactNode,
+  title: React.ReactNode;
   /** If user changes checkboxes, the component will send back column array with this property altered. */
-  isShown?: boolean,
+  isShown?: boolean;
   /** Set to false if the column should be hidden initially */
-  isShownByDefault: boolean,
+  isShownByDefault: boolean;
   /** The checkbox will be disabled, this is applicable to columns which should not be toggleable by user */
-  isUntoggleable?: boolean
+  isUntoggleable?: boolean;
 }
 
 /** extends ModalProps */
 export interface ColumnManagementModalProps extends Omit<ModalProps, 'ref' | 'children'> {
   /** Flag to show the modal */
-  isOpen?: boolean,
+  isOpen?: boolean;
   /** Invoked when modal visibility is changed */
-  onClose?: (event: KeyboardEvent | MouseEvent) => void,
+  onClose?: (event: KeyboardEvent | React.MouseEvent) => void;
   /** Current column state */
-  appliedColumns: ColumnManagementModalColumn[],
+  appliedColumns: ColumnManagementModalColumn[];
   /** Invoked with new column state after save button is clicked */
-  applyColumns: (newColumns: ColumnManagementModalColumn[]) => void,
+  applyColumns: (newColumns: ColumnManagementModalColumn[]) => void;
   /* Modal description text */
-  description?: string,
+  description?: string;
   /* Modal title text */
-  title?: string,
+  title?: string;
   /** Custom OUIA ID */
-  ouiaId?: string | number,
+  ouiaId?: string | number;
 }
 
 const ColumnManagementModal: FunctionComponent<ColumnManagementModalProps> = (
diff --git a/packages/module/src/ContentHeader/ContentHeader.tsx b/packages/module/src/ContentHeader/ContentHeader.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, PropsWithChildren, FunctionComponent } from 'react';
+import type { PropsWithChildren, FunctionComponent } from 'react';
 import {
   Flex,
   FlexItem,
@@ -24,19 +24,19 @@ export interface PageHeaderLinkProps extends ButtonProps {
 
 export interface ContentHeaderProps {
   /** Title for content header */
-  title: ReactNode;
+  title: React.ReactNode;
   /** Optional subtitle for content header */
-  subtitle?: ReactNode;
+  subtitle?: React.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?: ReactNode;
+  icon?: React.ReactNode;
   /** Optional label for content header (appears to the right of the content header's title) */
-  label?: ReactNode;
+  label?: React.ReactNode;
   /** Breadcrumbs component */
-  breadcrumbs?: ReactNode;
+  breadcrumbs?: React.ReactNode;
   /** Menu that appears to the far right of the title */
-  actionMenu?: ReactNode;
+  actionMenu?: React.ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
 }
diff --git a/packages/module/src/ErrorBoundary/ErrorBoundary.tsx b/packages/module/src/ErrorBoundary/ErrorBoundary.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, FunctionComponent } from 'react';
+import type { 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?: ReactNode;
+  headerTitle?: React.ReactNode;
   /** Indicates if the error is silent */
   silent?: boolean;
   /** The title text to display with the error */
-  errorTitle?: ReactNode;
+  errorTitle?: React.ReactNode;
   /** The description text to display with the error */
-  errorDescription?: ReactNode;
+  errorDescription?: React.ReactNode;
   /** The default description text to display with the error if no errorDescription is provided */
-  defaultErrorDescription?: ReactNode;
+  defaultErrorDescription?: React.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?: ReactNode;
+  children?: React.ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
   /** The heading level to use on the header title, default is h1 */
diff --git a/packages/module/src/ErrorState/ErrorState.tsx b/packages/module/src/ErrorState/ErrorState.tsx
@@ -10,7 +10,7 @@ import {
   StackItem, 
 } from '@patternfly/react-core';
 import { createUseStyles } from 'react-jss'
-import type { ReactNode, FunctionComponent } from 'react';
+import type { 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?: ReactNode;
+  titleText?: React.ReactNode;
   /** A description of the error, if no body text is provided then it will be set to the defaultBodyText. */
-  bodyText?: ReactNode;
+  bodyText?: React.ReactNode;
   /** A default description of the error used if no errorDescription is provided. */
-  defaultBodyText?: ReactNode;
+  defaultBodyText?: React.ReactNode;
   /** Custom footer content */
-  customFooter?: ReactNode;
+  customFooter?: React.ReactNode;
   /** ErrorState OUIA ID */
   ouiaId?: string | number;
   /** Status of the error message. */
diff --git a/packages/module/src/LogSnippet/LogSnippet.tsx b/packages/module/src/LogSnippet/LogSnippet.tsx
@@ -1,12 +1,12 @@
-import type { ReactNode, FunctionComponent } from 'react';
+import type { 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?: ReactNode;
+  logSnippet?: React.ReactNode;
   /** Message to appear above the log snippet */
-  message: string | ReactNode;
+  message: string | React.ReactNode;
   /** Log snippet alert variant */
   variant?: AlertVariant;
   /** Custom OUIA ID */
diff --git a/packages/module/src/Maintenance/Maintenance.tsx b/packages/module/src/Maintenance/Maintenance.tsx
@@ -1,27 +1,27 @@
-import type { ReactNode, FunctionComponent } from 'react';
+import type { 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?: ReactNode;
+    titleText?: React.ReactNode;
     /** Custom body text */
-    bodyText?: ReactNode;
+    bodyText?: React.ReactNode;
     /** A default bodyText used if no bodyText is provided */
-    defaultBodyText?: ReactNode;
+    defaultBodyText?: React.ReactNode;
     /** Start time in a specific time zone */
-    startTime?: ReactNode;
+    startTime?: React.ReactNode;
     /** End time in a specific time zone */
-    endTime?: ReactNode;
+    endTime?: React.ReactNode;
     /** Time zone specification */
-    timeZone?: ReactNode;
+    timeZone?: React.ReactNode;
     /** Information link */
     redirectLinkUrl?: string;
     /** Information link title */
-    redirectLinkText?: ReactNode;
+    redirectLinkText?: React.ReactNode;
     /** Custom footer content */
-    customFooter?: ReactNode;
+    customFooter?: React.ReactNode;
     /** Custom OUIA ID */
     ouiaId?: string | number;  }
 
diff --git a/packages/module/src/MissingPage/MissingPage.tsx b/packages/module/src/MissingPage/MissingPage.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, FunctionComponent } from 'react';
+import type { 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?: ReactNode;
+  toHomePageText?: React.ReactNode;
   /** The title for the invalid object message */
-  titleText?: ReactNode;
+  titleText?: React.ReactNode;
   /** The body text for the invalid object message */
-  bodyText?: ReactNode;
+  bodyText?: React.ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
 }
diff --git a/packages/module/src/MultiContentCard/MultiContentCard.tsx b/packages/module/src/MultiContentCard/MultiContentCard.tsx
@@ -1,4 +1,4 @@
-import type { ReactElement, ReactNode, FunctionComponent } from 'react';
+import type { ReactElement, FunctionComponent } from 'react';
 import { isValidElement, useState, Fragment } from 'react';
 import {
   Card,
@@ -21,21 +21,21 @@ export type MultiContentCardDividerVariant = typeof MultiContentCardDividerVaria
 
 export interface MutliContentCardProps {
   /** Card element to be displayed as a content */
-  content: ReactElement;
+  content: React.ReactElement;
   /** Allows adding divider on the left/right from the card */
   dividerVariant?: MultiContentCardDividerVariant;
 }
 
 /** extends CardProps */
 export interface MultiContentCardProps extends Omit<CardProps, 'children' | 'title'> {
   /** Cards to be displayed as a content */
-  cards?: (ReactElement | MutliContentCardProps)[];
+  cards?: (React.ReactElement | MutliContentCardProps)[];
   /** Actions to be displayed in the expandable section */
-  actions?: ReactElement;
+  actions?: React.ReactElement;
   /** Toggle text for the expandable section */
-  toggleText?: ReactNode;
+  toggleText?: React.ReactNode;
   /** Toggle content for the expandable section */
-  toggleContent?: ReactElement;
+  toggleContent?: React.ReactElement;
   /** When set to true, all content cards will be separated with dividers */
   withDividers?: boolean;
   /** Indicates whether the card is expandable */
diff --git a/packages/module/src/PageHeader/PageHeader.tsx b/packages/module/src/PageHeader/PageHeader.tsx
@@ -1,4 +1,4 @@
-import type { PropsWithChildren, ReactNode, FunctionComponent } from 'react';
+import type { FunctionComponent } from 'react';
 import {
   Button,
   ButtonProps,
@@ -23,25 +23,25 @@ export interface PageHeaderLinkProps extends ButtonProps {
   isExternal?: boolean;
 }
 
-export interface PageHeaderProps extends PropsWithChildren {
+export interface PageHeaderProps extends React.PropsWithChildren {
   /** Title for page header */
-  title?: ReactNode;
+  title?: React.ReactNode;
   /** Subtitle for page header */
-  subtitle?: ReactNode;
+  subtitle?: React.ReactNode;
   /** Optional link below subtitle */
   linkProps?: PageHeaderLinkProps;
   /** Optional icon for page header (appears to the left of the page header's title with a divider) */
-  icon?: ReactNode;
+  icon?: React.ReactNode;
   /** Optional label for page header (appears to the right of the page header's title) */
-  label?: ReactNode;
+  label?: React.ReactNode;
   /** Breadcrumbs component */
-  breadcrumbs?: ReactNode;
+  breadcrumbs?: React.ReactNode;
   /** Menu that appears to the far right of the title */
-  actionMenu?: ReactNode;
+  actionMenu?: React.ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
   /** Child nodes */
-  children?: ReactNode;
+  children?: React.ReactNode;
   /** Classname for the h1 element */
   headingClassname?: string;
 }
diff --git a/packages/module/src/ResponsiveAction/ResponsiveAction.tsx b/packages/module/src/ResponsiveAction/ResponsiveAction.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, FunctionComponent } from 'react';
+import type { FunctionComponent } from 'react';
 import { ButtonProps } from '@patternfly/react-core';
 
 /** extends ButtonProps */
@@ -10,7 +10,7 @@ export interface ResponsiveActionProps extends ButtonProps {
   /** Key for the action */
   key?: string;
   /** Action label */
-  children: ReactNode;
+  children: React.ReactNode;
 };
 
 // This component is only used declaratively - rendering ishandled by ResponsiveActions
diff --git a/packages/module/src/ResponsiveActions/ResponsiveActions.tsx b/packages/module/src/ResponsiveActions/ResponsiveActions.tsx
@@ -11,7 +11,7 @@ export interface ResponsiveActionsProps extends Omit<OverflowMenuProps, 'ref' |
   /** Custom OUIA ID */
   ouiaId?: string;
   /** Child actions to be displayed */
-  children: ReactNode;
+  children: React.ReactNode;
 }
 
 export const ResponsiveActions: FunctionComponent<ResponsiveActionsProps> = ({ ouiaId = 'ResponsiveActions', breakpoint = 'lg', children, ...props }: ResponsiveActionsProps) => {
diff --git a/packages/module/src/ServiceCard/ServiceCard.tsx b/packages/module/src/ServiceCard/ServiceCard.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, ReactElement, FunctionComponent } from 'react';
+import type { FunctionComponent } from 'react';
 import { Card, CardBody, CardFooter, CardHeader, CardProps, Content, ContentVariants, Flex, FlexItem } from '@patternfly/react-core';
 import { HelperText } from '@patternfly/react-core/dist/dynamic/components/HelperText';
 import { HelperTextItem } from '@patternfly/react-core/dist/dynamic/components/HelperText';
@@ -10,15 +10,15 @@ export interface ServiceCardProps extends CardProps {
   /** Service card title */
   title: string;
   /** Service card subtitle */
-  subtitle?: ReactNode;
+  subtitle?: React.ReactNode;
   /** Service card description */
-  description: ReactNode;
+  description: React.ReactNode;
   /** Service card icon */
-  icon: ReactNode;
+  icon: React.ReactNode;
   /** Optional Service card helper text*/
-  helperText?: ReactNode;
+  helperText?: React.ReactNode;
   /** Optional footer */
-  footer?: ReactElement | null;
+  footer?: React.ReactElement | null;
   /** Optional custom OUIA ID */
   ouiaId?: string | number;
   /** Optional flag modifying the card header layout */
diff --git a/packages/module/src/Severity/Severity.tsx b/packages/module/src/Severity/Severity.tsx
@@ -1,4 +1,4 @@
-import type { HTMLAttributes, DetailedHTMLProps, ReactNode, FunctionComponent } from 'react';
+import type { FunctionComponent } from 'react';
 import { Fragment, useMemo } from 'react';
 import {
   SeverityCriticalIcon,
@@ -38,11 +38,11 @@ export const SeverityType = {
 
 export type SeverityType = (typeof SeverityType)[keyof typeof SeverityType];
 
-export interface SeverityProps extends DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement> {
+export interface SeverityProps extends React.DetailedHTMLProps<React.HTMLAttributes<HTMLElement>, HTMLElement> {
   /** Determines a variant of displayed severity */
   severity: SeverityType;
   /** Label displayed next to the severity */
-  label: ReactNode;
+  label: React.ReactNode;
   /** Option to hide the label */
   labelHidden?: boolean;
   /** Custom className */
diff --git a/packages/module/src/Shortcut/Shortcut.tsx b/packages/module/src/Shortcut/Shortcut.tsx
@@ -1,4 +1,4 @@
-import type { ReactNode, FunctionComponent } from 'react';
+import type { FunctionComponent } from 'react';
 import { MouseIcon } from '@patternfly/react-icons';
 import { Label } from '@patternfly/react-core';
 import { createUseStyles } from 'react-jss';
@@ -8,7 +8,7 @@ export interface ShortcutProps {
   /** Array of shortcut keys */
   keys: string[];
   /** Shortcut description */
-  description?: ReactNode;
+  description?: React.ReactNode;
   /** Indicates whether symbols should be displayed for certain keys */
   showSymbols?: boolean;
   /** Show hover in the shortcut */
diff --git a/packages/module/src/SkeletonTable/SkeletonTable.tsx b/packages/module/src/SkeletonTable/SkeletonTable.tsx
@@ -15,7 +15,7 @@ export interface SkeletonTableProps
   /** The number of rows the skeleton table should contain */
   rowsCount?: number;
   /** Any captions that should be added to the table */
-  caption?: ReactNode;
+  caption?: React.ReactNode;
   /** Custom OUIA ID */
   ouiaId?: string | number;
   /** Flag indicating if the table is selectable */
diff --git a/packages/module/src/StaleDataWarning/StaleDataWarning.tsx b/packages/module/src/StaleDataWarning/StaleDataWarning.tsx
diff --git a/packages/module/src/Status/Status.tsx b/packages/module/src/Status/Status.tsx
diff --git a/packages/module/src/UnauthorizedAccess/UnauthorizedAccess.tsx b/packages/module/src/UnauthorizedAccess/UnauthorizedAccess.tsx
diff --git a/packages/module/src/UnavailableContent/UnavailableContent.tsx b/packages/module/src/UnavailableContent/UnavailableContent.tsx
diff --git a/packages/module/src/WarningModal/WarningModal.tsx b/packages/module/src/WarningModal/WarningModal.tsx
