Skip to content
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

feat(QuickResponse): Add active and selected states #403

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

rebeccaalpert
Copy link
Member

@rebeccaalpert rebeccaalpert commented Jan 9, 2025

Added two additional visual states to quick response labels. Only one option can be selected per block. Unlike the actions (thumbs up/down), selection persists if you click somewhere else.

State Before After
Default Screenshot 2025-01-09 at 3 33 58 PM Screenshot 2025-01-09 at 3 33 21 PM
Hover Screenshot 2025-01-09 at 3 34 07 PM Screenshot 2025-01-09 at 3 34 02 PM
Active Screenshot 2025-01-09 at 3 34 07 PM Screenshot 2025-01-09 at 3 33 30 PM
Selected Screenshot 2025-01-09 at 3 34 07 PM Screenshot 2025-01-09 at 3 33 36 PM

Click on some buttons here to see them: https://chatbot-pr-chatbot-403.surge.sh/patternfly-ai/chatbot/messages#messages-with-quick-responses

@patternfly-build
Copy link

patternfly-build commented Jan 9, 2025

@rebeccaalpert rebeccaalpert linked an issue Jan 9, 2025 that may be closed by this pull request
@edonehoo
Copy link
Contributor

edonehoo commented Jan 9, 2025

Could we change the first sentence in the example to mention that you can't select more than one?

You can offer convenient, clickable responses to messages in the form of quick actions. Quick actions are PatternFly labels in a label group, configured to display up to 5 visible labels. Only 1 response can be selected at a time.

Copy link

@kaylachumley kaylachumley left a comment

Choose a reason for hiding this comment

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

this looks good!

@rebeccaalpert
Copy link
Member Author

All set @edonehoo! 🌈

Copy link
Contributor

@edonehoo edonehoo left a comment

Choose a reason for hiding this comment

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

@nicolethoen nicolethoen merged commit 1bc43c2 into patternfly:main Jan 10, 2025
5 checks passed
Copy link

🎉 This PR is included in version 2.2.0-prerelease.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Add additional quick response states
5 participants