From 1dd0bb48d7bced4798b42754438498067b3d4e91 Mon Sep 17 00:00:00 2001 From: Rebecca Alpert Date: Tue, 21 Jul 2026 10:02:09 -0400 Subject: [PATCH 1/2] fix(icons): Replace with RH Ui icons --- .../src/FileDetailsLabel/FileDetailsLabel.tsx | 4 ++-- packages/module/src/ImagePreview/ImagePreview.tsx | 4 ++-- .../Message/CodeBlockMessage/CodeBlockMessage.tsx | 5 ++--- packages/module/src/Message/Message.test.tsx | 7 ++++++- .../src/Message/QuickResponse/QuickResponse.tsx | 4 ++-- .../src/Message/QuickStarts/QuickStartTile.tsx | 15 +++++++++------ packages/module/src/MessageBar/AttachButton.tsx | 4 ++-- .../module/src/MessageBar/MicrophoneButton.tsx | 9 ++------- packages/module/src/MessageBox/JumpButton.tsx | 8 ++------ 9 files changed, 29 insertions(+), 31 deletions(-) diff --git a/packages/module/src/FileDetailsLabel/FileDetailsLabel.tsx b/packages/module/src/FileDetailsLabel/FileDetailsLabel.tsx index 907a07c46..844062512 100644 --- a/packages/module/src/FileDetailsLabel/FileDetailsLabel.tsx +++ b/packages/module/src/FileDetailsLabel/FileDetailsLabel.tsx @@ -2,7 +2,7 @@ import { PropsWithChildren } from 'react'; import { Button, Label, LabelProps } from '@patternfly/react-core'; import FileDetails from '../FileDetails'; import { Spinner } from '@patternfly/react-core'; -import { TimesIcon } from '@patternfly/react-icons'; +import { RhMicronsCloseIcon } from '@patternfly/react-icons'; export interface FileDetailsLabelProps extends Omit { /** Name of file, including extension */ @@ -40,7 +40,7 @@ export const FileDetailsLabel = ({ spinnerTestId, fileSize, hasTruncation = true, - closeButtonIcon = , + closeButtonIcon = , ...props }: PropsWithChildren) => { const handleClose = (event) => { diff --git a/packages/module/src/ImagePreview/ImagePreview.tsx b/packages/module/src/ImagePreview/ImagePreview.tsx index 62a3aeaa0..398d64420 100644 --- a/packages/module/src/ImagePreview/ImagePreview.tsx +++ b/packages/module/src/ImagePreview/ImagePreview.tsx @@ -20,7 +20,7 @@ import { import { ChatbotDisplayMode } from '../Chatbot'; import ChatbotModal, { ChatbotModalProps } from '../ChatbotModal'; import FileDetailsLabel, { FileDetailsLabelProps } from '../FileDetailsLabel'; -import { TrashIcon } from '@patternfly/react-icons'; +import { RhUiTrashFillIcon } from '@patternfly/react-icons'; export interface ImagePreviewProps extends Omit { /** Class applied to modal */ @@ -121,7 +121,7 @@ const ImagePreview: FunctionComponent = ({ fileSize={images[page - 1].fileSize} hasTruncation={false} onClose={onCloseFileDetailsLabel} - closeButtonIcon={} + closeButtonIcon={} {...fileDetailsLabelProps} /> diff --git a/packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.tsx b/packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.tsx index bd83fa315..b3694a4bc 100644 --- a/packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.tsx +++ b/packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.tsx @@ -17,9 +17,8 @@ import { getUniqueId } from '@patternfly/react-core'; -import { CheckIcon } from '@patternfly/react-icons/dist/esm/icons/check-icon'; import { css } from '@patternfly/react-styles'; -import { RhUiCopyFillIcon } from '@patternfly/react-icons'; +import { RhMicronsCheckmarkIcon, RhUiCopyFillIcon } from '@patternfly/react-icons'; export interface CodeBlockMessageProps { /** Content rendered in code block */ @@ -142,7 +141,7 @@ const CodeBlockMessage = ({ className="pf-chatbot__button--copy" onClick={(event) => handleCopy(event, children)} > - {copied ? : } + {copied ? : } diff --git a/packages/module/src/Message/Message.test.tsx b/packages/module/src/Message/Message.test.tsx index 2900c1687..5609e1fec 100644 --- a/packages/module/src/Message/Message.test.tsx +++ b/packages/module/src/Message/Message.test.tsx @@ -26,7 +26,12 @@ jest.mock('@patternfly/react-icons', () => ({ RhUiExportFillIcon: () =>
RhUiExportFillIcon
, RhMicronsInformationFillIcon: () =>
RhMicronsInformationFillIcon
, RhUiBackwardsIcon: () =>
RhUiBackwardsIcon
, - RhMicronsExternalLinkIcon: () =>
RhMicronsExternalLinkIcon
+ RhMicronsExternalLinkIcon: () =>
RhMicronsExternalLinkIcon
, + RhMicronsCheckmarkIcon: () =>
RhMicronsCheckmarkIcon
, + RhMicronsCloseIcon: () =>
RhMicronsCloseIcon
, + RhUiBookmarkIcon: () =>
RhUiBookmarkIcon
, + RhUiClockIcon: () =>
RhUiClockIcon
, + RhUiRocketFillIcon: () =>
RhUiRocketFillIcon
})); const ALL_ACTIONS = [ diff --git a/packages/module/src/Message/QuickResponse/QuickResponse.tsx b/packages/module/src/Message/QuickResponse/QuickResponse.tsx index 033cdd9fa..5c39ec4b6 100644 --- a/packages/module/src/Message/QuickResponse/QuickResponse.tsx +++ b/packages/module/src/Message/QuickResponse/QuickResponse.tsx @@ -1,7 +1,7 @@ import type { FunctionComponent } from 'react'; import { useState } from 'react'; import { Label, LabelGroup, LabelGroupProps, LabelProps } from '@patternfly/react-core'; -import { CheckIcon } from '@patternfly/react-icons'; +import { RhMicronsCheckmarkIcon } from '@patternfly/react-icons'; import { css } from '@patternfly/react-styles'; export interface QuickResponse extends Omit { @@ -42,7 +42,7 @@ export const QuickResponse: FunctionComponent = ({ {quickResponses.map(({ id, onClick, content, className, ...props }: QuickResponse) => (
From cb9130eee0f87ca95f607085dccc464e84ccd707 Mon Sep 17 00:00:00 2001 From: Rebecca Alpert Date: Tue, 21 Jul 2026 14:43:28 -0400 Subject: [PATCH 2/2] Address feedback --- .../examples/Messages/explore-pipeline-quickstart.ts | 3 --- packages/module/src/ImagePreview/ImagePreview.tsx | 6 +++++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/packages/module/patternfly-docs/content/extensions/chatbot/examples/Messages/explore-pipeline-quickstart.ts b/packages/module/patternfly-docs/content/extensions/chatbot/examples/Messages/explore-pipeline-quickstart.ts index 4dd0faa69..4699b231b 100644 --- a/packages/module/patternfly-docs/content/extensions/chatbot/examples/Messages/explore-pipeline-quickstart.ts +++ b/packages/module/patternfly-docs/content/extensions/chatbot/examples/Messages/explore-pipeline-quickstart.ts @@ -1,5 +1,3 @@ -import userAvatar from './user_avatar.svg'; - export const explorePipelinesQuickStart = { apiVersion: 'console.openshift.io/v1', kind: 'QuickStarts', @@ -10,7 +8,6 @@ export const explorePipelinesQuickStart = { version: 4.7, displayName: `Installing the Pipelines Operator`, durationMinutes: 10, - icon: userAvatar, description: `Install the OpenShift® Pipelines Operator to build Pipelines using Tekton.`, prerequisites: [''], introduction: `OpenShift® Pipelines is a cloud-native, continuous integration and continuous delivery (CI/CD) solution based on Kubernetes resources. It uses Tekton building blocks to automate deployments across multiple Kubernetes distributions by abstracting away the underlying implementation details. diff --git a/packages/module/src/ImagePreview/ImagePreview.tsx b/packages/module/src/ImagePreview/ImagePreview.tsx index 398d64420..01d6b59b0 100644 --- a/packages/module/src/ImagePreview/ImagePreview.tsx +++ b/packages/module/src/ImagePreview/ImagePreview.tsx @@ -121,7 +121,11 @@ const ImagePreview: FunctionComponent = ({ fileSize={images[page - 1].fileSize} hasTruncation={false} onClose={onCloseFileDetailsLabel} - closeButtonIcon={} + closeButtonIcon={ + + + + } {...fileDetailsLabelProps} />