Skip to content

Conversation

@its-mitesh-kumar
Copy link
Member

@its-mitesh-kumar its-mitesh-kumar commented Jan 15, 2026

Summary

Refactors the ToolCallContent component to use PatternFly components instead of Material-UI, following the PatternFly AI Chatbot design patterns.

UI after changes

Light Mode

Screen.Recording.2026-01-16.at.3.19.28.PM.mov

Dark mode

Screenshot 2026-01-16 at 3 25 09 PM (2)

Fixes

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

Signed-off-by: its-mitesh-kumar <itsmiteshkumar98@gmail.com>
Signed-off-by: its-mitesh-kumar <itsmiteshkumar98@gmail.com>
@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Jan 15, 2026

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-lightspeed workspaces/lightspeed/plugins/lightspeed patch v1.1.2

@its-mitesh-kumar
Copy link
Member Author

its-mitesh-kumar commented Jan 15, 2026

@aprilma419 Please have a look at screen recording for Tool Call .
Cc: @debsmita1 @karthikjeeyar

@aprilma419
Copy link

@its-mitesh-kumar Does the copy icon copy anything from the tool call? If not, I think we can just remove it
Screenshot 2026-01-16 at 10 48 14

Copy link
Member

@ciiay ciiay left a comment

Choose a reason for hiding this comment

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

Hi @its-mitesh-kumar , the yarn test fails for me with your branch. Can you verify if it works for you?

@its-mitesh-kumar
Copy link
Member Author

@its-mitesh-kumar Does the copy icon copy anything from the tool call? If not, I think we can just remove it =

@aprilma419 Copy icon is being used to copy the response.

@aprilma419
Copy link

@its-mitesh-kumar Does the copy icon copy anything from the tool call? If not, I think we can just remove it =

@aprilma419 Copy icon is being used to copy the response.

@its-mitesh-kumar Okay, if so, then I think the copy icon needs to be aligned in the same horizontal row with the Response title in the UI like this

Screenshot 2026-01-16 at 14 08 30

@its-mitesh-kumar
Copy link
Member Author

@its-mitesh-kumar Okay, if so, then I think the copy icon needs to be aligned in the same horizontal row with the Response title in the UI like this

@aprilma419 Updated the Screen Recording in the description.

@sonarqubecloud
Copy link

@aprilma419
Copy link

@its-mitesh-kumar The recording is only 1 second long...

The screenshot of the dark mode looks good to me

Copy link
Contributor

@HusneShabbir HusneShabbir left a comment

Choose a reason for hiding this comment

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

/lgtm
Screenshot 2026-01-16 at 4 47 26 PM
Screenshot 2026-01-16 at 4 47 41 PM

Copy link
Member

@karthikjeeyar karthikjeeyar left a comment

Choose a reason for hiding this comment

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

/approve
/lgtm

@karthikjeeyar karthikjeeyar merged commit c5606c3 into redhat-developer:main Jan 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants