improvement(tables): show a tooltip on truncated column headers - #6371
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview
It replaces the previous truncating Reviewed by Cursor Bugbot for commit f98a1cd. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR introduces a reusable clipped-header label that displays the full text in a floating tooltip and applies it to column and workflow-group headers.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/headers/header-label.tsx | Adds the reusable truncated-label component and conditionally displayed floating tooltip. |
| apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/headers/column-header-menu.tsx | Uses HeaderLabel for both read-only and interactive column-header labels. |
| apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/headers/workflow-group-meta-cell.tsx | Uses HeaderLabel for workflow and enrichment group names. |
Reviews (2): Last reviewed commit: "chore(tables): use absolute import for H..." | Re-trigger Greptile
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit f98a1cd. Configure here.
Summary
isTextClipped, so an untruncated header never shows oneTableGrid), plus the workflow/enrichment group headerHeaderLabelcomponent replaces the three hand-rolled truncating spansType of Change
Testing
Tested manually.
bun run type-check,bun run lint, andbun run check:audits(22 audits) pass.Checklist