Skip to content

Conversation

tenphi
Copy link
Member

@tenphi tenphi commented Oct 7, 2025

Note

Introduce controlled search (searchValue/onSearchChange) and the ability to disable internal filtering (filter=false), wired through FilterPicker with tests and stories.

  • FilterListBox:
    • Add controlled search via searchValue and onSearchChange props.
    • Allow disabling internal filtering with filter=false; fallback to no-op filter when disabled.
    • Refactor internal search handling (handleSearchChange), ensure Escape clears controlled/uncontrolled search.
  • FilterPicker:
    • Plumb searchValue and onSearchChange to underlying FilterListBox.
    • Preserve existing props; no behavior change unless new props used.
  • Tests/Stories:
    • Add test for controlled search with filter={false} and external item filtering in FilterListBox.test.tsx.
    • Add Storybook "ExternalFiltering" demonstrating filter={false} and controlled search patterns.

Written by Cursor Bugbot for commit 4c67e37. This will update automatically on new commits. Configure here.

@changeset-bot
Copy link

changeset-bot bot commented Oct 7, 2025

🦋 Changeset detected

Latest commit: 4c67e37

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cube-dev/ui-kit Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Oct 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
cube-ui-kit Ready Ready Preview Comment Oct 7, 2025 3:18pm
cube-ui-kit-cursor Ready Ready Preview Comment Oct 7, 2025 3:18pm

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

📦 NPM canary release

Deployed canary version 0.0.0-canary-8dfe8a6.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

🧪 Storybook is successfully deployed!

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

🏋️ Size limit report

Name Size Passed?
All 299.6 KB (+0.03% 🔺) Yes 🎉
Tree shaking (just a Button) 32.07 KB (0% 🟰) Yes 🎉
Tree shaking (just an Icon) 19.39 KB (0% 🟰) Yes 🎉

Click here if you want to find out what is changed in this build

@tenphi tenphi merged commit 13fb450 into main Oct 7, 2025
14 checks passed
@tenphi tenphi deleted the fix-filterpicket-items branch October 7, 2025 15:51
@github-actions github-actions bot mentioned this pull request Oct 7, 2025
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.

1 participant