Skip to content

feat: add filterPlaceholder support to DefaultColumnFilter#93

Open
Abhishek9639 wants to merge 1 commit intocertego:developfrom
Abhishek9639:feature/filter-placeholder-support
Open

feat: add filterPlaceholder support to DefaultColumnFilter#93
Abhishek9639 wants to merge 1 commit intocertego:developfrom
Abhishek9639:feature/filter-placeholder-support

Conversation

@Abhishek9639
Copy link
Copy Markdown

Description

Added support for an optional filterPlaceholder prop in [DefaultColumnFilter] component.

This allows consumers (like IntelOwl) to customize the placeholder text for specific columns (e.g., showing "Search ID" instead of the default "Search keyword..").

Changes

  • [DefaultColumnFilter] now accepts filterPlaceholder from the column definition
  • If filterPlaceholder is provided, it is used as the input placeholder
  • If not provided, falls back to the default "Search keyword.." (backward compatible)

Related

@Abhishek9639 Abhishek9639 changed the base branch from main to develop March 20, 2026 20:25
@Abhishek9639
Copy link
Copy Markdown
Author

Greetings @mlodic,
I've created this PR to add the filterPlaceholder property at the library level as we discussed for intelowlproject/IntelOwl#1553.
Could you please take a look when you have a moment?
Thank you

@mlodic
Copy link
Copy Markdown
Contributor

mlodic commented Mar 22, 2026

I don't have access to this repo anymore, I need @drosetti or @fgibertoni to merge this

@Abhishek9639
Copy link
Copy Markdown
Author

Greetings @drosetti or @fgibertoni,

Could you please review and merge this PR when you have a moment?
This is the companion library change needed for intelowlproject/IntelOwl#3504.

@mlodic has already reviewed the approach and is waiting on this certego-ui PR to be merged first.

Thank you

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.

2 participants