Skip to content

improvement(tables): show a tooltip on truncated column headers - #6371

Merged
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/table-column-hover
Aug 7, 2026
Merged

improvement(tables): show a tooltip on truncated column headers#6371
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/table-column-hover

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • Column headers in the table grid now reveal their full name in an emcn floating tooltip when the text is clipped — gated on isTextClipped, so an untruncated header never shows one
  • Covers both surfaces: the standalone table page and the embedded table view in the mothership chat page (same TableGrid), plus the workflow/enrichment group header
  • New HeaderLabel component replaces the three hand-rolled truncating spans

Type of Change

  • Improvement

Testing

Tested manually. bun run type-check, bun run lint, and bun run check:audits (22 audits) pass.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 7, 2026 5:53am

Request Review

@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Presentation-only change in table grid headers with no API, auth, or data logic touched.

Overview
Truncated table column headers and workflow/enrichment group names now show the full label in a floating tooltip on hover, using a new shared HeaderLabel component.

HeaderLabel truncates with an ellipsis and only opens the tooltip when isTextClipped says the text is actually clipped, matching the pattern used elsewhere in the app.

It replaces the previous truncating <span>s in column-header-menu (read-only and interactive column headers) and workflow-group-meta-cell (group header name).

Reviewed by Cursor Bugbot for commit f98a1cd. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The 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.

  • Adds HeaderLabel using the shared floating-tooltip utilities and clipping predicate.
  • Replaces three hand-written truncating spans across the table-grid headers.
  • Updates both consumers to use absolute alias imports.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

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

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ 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.

@TheodoreSpeaks
TheodoreSpeaks merged commit e6a17b0 into staging Aug 7, 2026
30 checks passed
@waleedlatif1
waleedlatif1 deleted the feat/table-column-hover branch August 7, 2026 17:03
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