-
Notifications
You must be signed in to change notification settings - Fork 29
feat(CodeBlockMessage/Message): Add expandable code block #556
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat(CodeBlockMessage/Message): Add expandable code block #556
Conversation
Preview: https://chatbot-pr-chatbot-556.surge.sh A11y report: https://chatbot-pr-chatbot-556-a11y.surge.sh |
54818d3
to
93da6cf
Compare
93da6cf
to
30e6135
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small thing! Can we update the link text to be sentence case?
packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.tsx
Outdated
Show resolved
Hide resolved
All set @edonehoo; thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! just 2 comments on quick variable swaps
packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.scss
Outdated
Show resolved
Hide resolved
packages/module/src/Message/CodeBlockMessage/CodeBlockMessage.scss
Outdated
Show resolved
Hide resolved
Co-authored-by: Erin Donehoo <[email protected]>
8e5df16
to
c977224
Compare
Thanks @kaylachumley! All set. |
Looks great; Thank you for adding this feature. |
This was a request from RHDH product manager - wanted to be able to make code blocks optionally expandable. They were fine with having it applied to all code blocks in message - they envision having it applied to all user messages by default.
Kayla wanted the truncated toggle applied by default - it sounds like design will have general PF migrate in that direction soon.
Docs: https://chatbot-pr-chatbot-556.surge.sh/patternfly-ai/chatbot/messages (go to user or bot message and select "expandable code" option from dropdown.)