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

Feature request: Quick response disable prop #443

Open
ibolton336 opened this issue Feb 12, 2025 · 2 comments · May be fixed by #500
Open

Feature request: Quick response disable prop #443

ibolton336 opened this issue Feb 12, 2025 · 2 comments · May be fixed by #500
Assignees
Labels
PF6 Applies to only the PF6 version
Milestone

Comments

@ibolton336
Copy link
Member

ibolton336 commented Feb 12, 2025

Would be great to be able to visually disable the quick response. Not seeing anything in the quickResponseContainerProps that would allow this currently.

Image
@ibolton336
Copy link
Member Author

ibolton336 commented Feb 12, 2025

Image

Update: found the disabled prop, but does not appear to have any visual effect. Will add a custom className prop here for now.

@rebeccaalpert rebeccaalpert added the PF6 Applies to only the PF6 version label Feb 13, 2025
@rebeccaalpert rebeccaalpert self-assigned this Feb 13, 2025
@rebeccaalpert rebeccaalpert moved this from Needs triage to Not started in PatternFly Issues Feb 13, 2025
@rebeccaalpert rebeccaalpert added this to the 2025.Q2 milestone Feb 27, 2025
@rebeccaalpert rebeccaalpert moved this from Not started to In Progress in PatternFly Issues Mar 28, 2025
@rebeccaalpert
Copy link
Member

rebeccaalpert commented Mar 28, 2025

Hey @ibolton336 - I had some time to dig into this and it looks like we already allow this. I'll add a demo, but this is how you do it:

<Message
      name="Bot"
      role="bot"
      avatar={patternflyAvatar}
      content="Did you clear your cache?"
      quickResponses={[
        { id: '1', content: 'Yes', onClick: () => alert('Clicked yes'), isDisabled: true },
        { id: '2', content: 'No', onClick: () => alert('Clicked no') }
      ]}
    />
Image

You can also pass any other label props down this way. Label group props have a separate prop in the props table.

@rebeccaalpert rebeccaalpert linked a pull request Mar 28, 2025 that will close this issue
@rebeccaalpert rebeccaalpert moved this from In Progress to PR Review in PatternFly Issues Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PF6 Applies to only the PF6 version
Projects
Status: PR Review
Development

Successfully merging a pull request may close this issue.

2 participants