Skip to content

fix(ChatbotConversationHistoryNav): Pass search input props - #630

Merged
rebeccaalpert merged 1 commit into
patternfly:mainfrom
rebeccaalpert:searchInput
Aug 4, 2025
Merged

rebeccaalpert merged 1 commit into
patternfly:mainfrom
rebeccaalpert:searchInput

Conversation

@rebeccaalpert

Copy link
Copy Markdown
Member

Allow for controlled inputs and other customizations by passing props down.

@patternfly-build

patternfly-build commented Jul 29, 2025

Copy link
Copy Markdown

expect(iconElement).toBeInTheDocument();
});

it('should be able to have controlled search input', () => {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just a nit based on how react tests are written, but maybe "Should be able to spread search input props when searchInputProps is passed" or something.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

All set! Thank you.

Allow for controlled inputs and other customizations by passing props down.
@rebeccaalpert

Copy link
Copy Markdown
Member Author

Just rebasing after Eric's merged.

@rebeccaalpert
rebeccaalpert merged commit b71c16d into patternfly:main Aug 4, 2025
7 checks passed
@github-actions

github-actions Bot commented Aug 4, 2025

Copy link
Copy Markdown

🎉 This PR is included in version 6.4.0-prerelease.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

rebeccaalpert added a commit to rebeccaalpert/virtual-assistant that referenced this pull request Oct 24, 2025
…ly#630)

Allow for controlled inputs and other customizations by passing props down.
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.

Search query is cleared when you toggle the conversation drawer.

3 participants