feat(mosaic): add shared table toolbar and search - #9915
Conversation
🦋 Changeset detectedLatest commit: 205297c The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types 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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (21)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
💤 Files with no reviewable changes (7)
Included review availability: 7 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 8 reviews per hour. 📝 WalkthroughWalkthroughThe table package adds Estimated code review effort: 3 (Moderate) | ~25 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to The shared table controls are ready to merge after normal checks; no blocking issue was identified. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 10 functions across 11 files. (3 skipped: 3 unsupported.)
Comment |
2a0b3f4 to
28ad4a8
Compare
1e2f875 to
43121c9
Compare
28ad4a8 to
3ce8983
Compare
43121c9 to
c9594de
Compare
c9594de to
205297c
Compare
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-google-signin
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/mosaic
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
Description
Adds
Table.Toolbar, controlledTable.Search, andTable.Cell noWrapto share table controls and date-cell styling. Search composes the existing InputGroup, Icon, and Button, and restores input focus when cleared.Uses the shared toolbar across all five searchable Mosaic tables: organization Members, Invitations, Requests, and the organization and user API key panels. Adds a working Swingset example and documentation. In Swingset, search and clear each table, then check pagination, selection, and focus after removing a row.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change