Skip to content

feat(filters): use chipTitle prop for filter chip category#633

Open
raswonders wants to merge 4 commits intopatternfly:mainfrom
raswonders:main
Open

feat(filters): use chipTitle prop for filter chip category#633
raswonders wants to merge 4 commits intopatternfly:mainfrom
raswonders:main

Conversation

@raswonders
Copy link
Copy Markdown
Contributor

Hey!

in our codebase we are sometimes in need to use different chip title than the title used in dropdown menu. This could be due to various reasons, sometimes dropdown title is just to wordy and takes too much space, other times its for compatibility with our previous solution.

What it does:

It introduces additional optional prop chipTitle which allows passing different name for chip than its used in filter dropdown.

Compatibility

Change is backward compatible when this new prop is omitted component behaves as before.

would you find this useful? Also If you'd agree I can patch other filter types as well.

Thanks!

@patternfly-build
Copy link
Copy Markdown

patternfly-build commented Apr 21, 2026

@karelhala
Copy link
Copy Markdown
Collaborator

@raswonders can you please also add example of the new chiptitle? And also, might be worth to add chipTitle prop to all filter variants.

@raswonders raswonders changed the title feat(DataViewTextFilter): use chipTitle prop for filter chip category feat(filters): use chipTitle prop for filter chip category May 4, 2026
@raswonders
Copy link
Copy Markdown
Contributor Author

@raswonders can you please also add example of the new chiptitle? And also, might be worth to add chipTitle prop to all filter variants.

I've implemented change in other filters as well. Regarding the example, I'm not sure what exactly you have in mind. I've extended FilteringExample and used chipTitle="Repo" right in there. Is this ok or you've imagined it differently?

@raswonders raswonders requested a review from karelhala May 4, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants