Skip to content

fix(MessageBar): Replace send button icon - #903

Open
gabipodolnikova wants to merge 1 commit into
patternfly:mainfrom
gabipodolnikova:pf-4462-replace-send-icon
Open

fix(MessageBar): Replace send button icon#903
gabipodolnikova wants to merge 1 commit into
patternfly:mainfrom
gabipodolnikova:pf-4462-replace-send-icon

Conversation

@gabipodolnikova

Copy link
Copy Markdown

Replace the PaperPlaneIcon used by SendButton with the RH Ui Send icon from @patternfly/react-icons.

This currently uses the prerelease dependency @patternfly/react-icons@6.6.1-prerelease.2, released after PF-4576. The dependency can be updated to the stable release without code changes once available.

Restore PatternFly's default plain icon-button interaction styles while preserving the brand color for the default state.

Fixes #880

Tests:

  • npx jest --runInBand packages/module/src/MessageBar/SendButton.test.tsx
  • npm run build -w @patternfly/chatbot
  • npx eslint packages/module/src/MessageBar/SendButton.tsx

@patternfly-build

patternfly-build commented Sep 11, 2026

Copy link
Copy Markdown

@rebeccaalpert rebeccaalpert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is 100% my fault - would you be willing to use the filled version (if it exists?): https://www.figma.com/design/MSr6kVEOuAxmPOkjg7x8PO/PatternFly-6%3A-Patterns-%26-Extensions?m=auto&node-id=5190-4959&t=kak1QKBUdZzkndwZ-1

Looking at Kayla's design, we want Rh UI Send Fill Icon instead. Everything else looks great to me. 100% my fault.

@kaylachumley kaylachumley left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for working on this. +1 on the comment about the rh filled send icon change.

Does this pr also include other icons in the message bar being reverted back to the standard icon button styling? If so, it looks like the attach plus icon needs to be changed from subtle icon color to regular icon color and same for the other variation of attach(paper clip icon). Thanks!

Image Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SendButton - Replace icon

4 participants