Skip to content

docs: full value on hover for truncated table cells#11296

Merged
igorlukanin merged 2 commits into
masterfrom
igor/cub-3299-docs-truncated-cell-tooltip
Jul 20, 2026
Merged

docs: full value on hover for truncated table cells#11296
igorlukanin merged 2 commits into
masterfrom
igor/cub-3299-docs-truncated-cell-tooltip

Conversation

@igorlukanin

@igorlukanin igorlukanin commented Jul 19, 2026

Copy link
Copy Markdown
Member

Summary

Documents that a truncated cell reveals its full value on hover:

  • Table chart (charts/chart-types/table.mdx) — one line next to the Word wrap column option, so the truncate/wrap behavior and its hover affordance are described together.
  • Results table (workbooks/querying-data.mdx) — one line in Running queries.

Test plan

  • Wording matches the shipped behavior (tooltip shows only when a cell is actually truncated and word wrap is off).
  • Mintlify build passes.

@igorlukanin
igorlukanin requested a review from keydunov as a code owner July 19, 2026 21:55
@mintlify

mintlify Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
cubed3 🟢 Ready View Preview Jul 19, 2026, 9:56 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@claude

claude Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Claude finished @igorlukanin's task in 25s —— View job


Code Review

Verdict: LGTM — 0 issues. Expand for details.
  • Read the diff on docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx
  • Verify wording, placement, and Mintlify conventions

Change

Adds one sentence at table.mdx:54, right below the Column field options table (which contains the Word wrap row), documenting that a cell truncated with an ellipsis reveals its full value on hover:

When a cell's value is too long for its column and word wrap is off, the value is truncated with an ellipsis. Hover over a truncated cell to see its full value in a tooltip.

Review

  • Placement — correctly co-located with the Word wrap option so the truncate/wrap pair and the hover affordance are described together, matching the PR summary. Not buried inside the table row (which would strain the two-column layout).
  • Wording — precise about the preconditions ("too long for its column" + "word wrap is off"), matches the described shipped behavior (tooltip only when actually truncated).
  • Style — plain prose paragraph, no MDX components needed; consistent with the rest of the file. No frontmatter, nav, or docs.json changes required for a text tweak.
  • Nits — none worth blocking on. If you wanted extra polish you could bold Word wrap on first mention to mirror how other option names are styled in body prose elsewhere in the file, but the current phrasing is fine.

No security, performance, or test-coverage concerns apply to a docs-only change. Waiting on the Mintlify preview build for the final visual check.

· Branch: `igor/cub-3299-docs-truncated-cell-tooltip`

@claude

claude Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Claude finished @igorlukanin's task in 47s —— View job


Code Review

Verdict: LGTM — 0 blocking issues, 1 observation. Expand for details.
  • Read the diff for both commits vs origin/master
  • Check wording, placement, Mintlify conventions, and CLAUDE.md style rules
  • Cross-check PR description against actual changes

Changes

  1. docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx:54 — one sentence added under the Column field options table, documenting that a truncated cell reveals its full value on hover when word wrap is off. Matches the PR summary exactly; placement is correct (co-located with the Word wrap option, not buried inside a table row).

  2. docs-mintlify/docs/explore-analyze/workbooks/querying-data.mdx — two additions:

    • Lines 27–28: parallel note about the truncated-cell tooltip in the workbook results context. Wording matches the table-chart version, which is good for consistency.
    • Lines 219–224: a new "Result freshness and provenance" section describing the leaf (freshness) and lightning-bolt (pre-aggregation) indicators next to each result, including hover labels, aging color shift, "Data age unknown" fallback, and the Query History click-through for authorized users.

Review notes

  • Wording — precise about preconditions in both truncation notes ("too long for its column"; the table version also correctly qualifies "word wrap is off"). Matches the shipped behavior described in the PR.
  • Style — plain prose, no MDX components needed. Consistent with the surrounding file. Bolded UI labels (Freshness, Pre-aggregation, Word wrap), inline links to /admin/monitoring/query-history and /docs/pre-aggregations, and root-relative paths all follow docs-mintlify/CLAUDE.md.
  • Nav — no docs.json change needed; both edits modify existing pages.
  • Scope observation (non-blocking) — the PR title, summary, and test-plan describe only the truncated-cell tooltip. The Result freshness and provenance section is a substantive new subsection (10 lines, new indicators, new click-through behavior) that isn't mentioned in the description. Consider either updating the PR summary to call it out so reviewers know to check that content, or splitting it into a separate PR. Either is fine; just flagging that the description undersells what shipped.
  • Security / performance / tests — N/A for docs-only text changes.

Nothing blocking. Waiting on the Mintlify preview build for the visual check.

· Branch: `igor/cub-3299-docs-truncated-cell-tooltip`

@igorlukanin igorlukanin changed the title docs: full value on hover for truncated table chart cells docs: full value on hover for truncated table cells Jul 20, 2026
@igorlukanin
igorlukanin merged commit 3284ee2 into master Jul 20, 2026
9 checks passed
@igorlukanin
igorlukanin deleted the igor/cub-3299-docs-truncated-cell-tooltip branch July 20, 2026 21:05
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