Skip to content

docs: document time-dimension drill-down in table charts#11244

Merged
igorlukanin merged 3 commits into
masterfrom
igor/cub-1500-document-time-drill-down
Jul 17, 2026
Merged

docs: document time-dimension drill-down in table charts#11244
igorlukanin merged 3 commits into
masterfrom
igor/cub-1500-document-time-drill-down

Conversation

@igorlukanin

Copy link
Copy Markdown
Member

Summary

Documents the new time-dimension drill-down in table charts: clicking a time cell that's shown at a granularity offers Drill down to a finer granularity, re-viewing the same measures scoped to the clicked period. Adds it to the table chart's "Cell menu" section, alongside the existing measure drill-down.

Test plan

  • Prose reconciled against the shipped UI (drill labels, granularity behavior, week handling).
  • Docs build passes.

@igorlukanin
igorlukanin requested a review from keydunov as a code owner July 13, 2026 22:51
@claude

claude Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

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


Review: docs — time-dimension drill-down

Docs-only change; prose reads cleanly and matches the style of the surrounding "Cell menu" section. No build/test impact.

Details — 0 blocking, 2 nits

Task list

  • Read PR diff (docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx, +25/-2)
  • Reconcile against surrounding sections and doc conventions
  • Note nits

What's good

  • Split into two ### subsections (measure vs. time dimension) under the existing Cell menu heading — fits the file's structure.
  • Behavior is explained concretely (year → quarter example, week/calendar caveat, return path).
  • Language matches naming conventions in docs-mintlify/CLAUDE.md (no product-name issues, second-person voice, plain MDX).

Nits (optional)

  1. table.mdx:177 — the granularity list is year, quarter, month, week, day, hour, or minute. If a cell can also be displayed by second (Cube supports second as a granularity), it's missing here; if not, ignore. Worth a quick reconcile against the shipped granularity picker.
  2. table.mdx:178-179 — "a cell shown by year offers Quarter, Month, Day, and finer" is slightly asymmetric with the following line ("a cell shown by quarter offers Month, Day, and finer") because the year example skips a level (no Month? actually it's listed). Re-reading, it's fine — just double-check that a year cell in the UI actually offers Hour and Minute as "finer" targets, since jumping year → hour is unusual. If the UI caps the drill depth per starting granularity, that's worth one sentence.

Neither blocks merging.

• Branch: `igor/cub-1500-document-time-drill-down`

@mintlify

mintlify Bot commented Jul 13, 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 13, 2026, 10:54 PM

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

Copy link
Copy Markdown
Member

Too verbose

@igorlukanin

Copy link
Copy Markdown
Member Author

Tightened it — the two subsections are now one short paragraph in 74dc466.

@claude

claude Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

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


Review: docs — time-dimension drill-down (tightened)

Docs-only tightening; reads well and addresses the "too verbose" note. Approved-in-spirit; no blocking issues.

Details — 0 blocking, 1 nit

Checklist

  • Re-read tightened diff (docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx, +15/-2)
  • Verify structure still fits the "Cell menu" section
  • Reconcile against Mintlify conventions

What's good

  • Collapses two ### subsections into one bulleted paragraph — much less scan cost, same information.
  • Keeps the concrete 2024 → Quarter example and the calendar-hierarchy caveat, which are the load-bearing bits.
  • Week caveat rewritten as a single sentence at the end; reads naturally as an exception rather than a full section.
  • Voice, MDX plainness, and second-person tone match docs-mintlify/CLAUDE.md.

Nit (optional)

  • table.mdx:178-179 — "Week is only ever offered when a cell is already shown by week" is technically correct but slightly circular on first read (why offer Week on a Week cell at all?). Consider: "Week is only offered as a drill target from a cell that is already shown by week — never from a coarser granularity." Not a blocker.
• Branch: `igor/cub-1500-document-time-drill-down`

@claude

claude Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

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


Review: docs — time-dimension drill-down (final)

Docs-only tightening. Reads cleanly, matches Mintlify conventions, and addresses the "too verbose" note. No blocking issues.

Details — 0 blocking, 0 nits (prior nit still stands, non-blocking)

Checklist

  • Re-read docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx (lines 161–179)
  • Verify structure fits the "Cell menu" section
  • Reconcile against docs-mintlify/CLAUDE.md conventions
  • Check prior review threads for staleness

What's good

  • One short paragraph with a two-bullet split (measure vs. time dimension) — much less scan cost than the earlier two-subsection version.
  • Load-bearing bits kept: concrete 2024 → Quarter example, "same measures" invariant, close-to-return path, and the week-straddles-months caveat.
  • Voice, second-person tone, and plain MDX all match docs-mintlify/CLAUDE.md.

Prior nit (still applicable, still non-blocking)

  • table.mdx:178 — "Week is never a drill target from a coarser granularity — a week cell drills only to Day and finer" reads fine now; the earlier circular phrasing is gone. No action needed.
• Branch: `igor/cub-1500-document-time-drill-down`

@igorlukanin
igorlukanin merged commit c73f67d into master Jul 17, 2026
9 checks passed
@igorlukanin
igorlukanin deleted the igor/cub-1500-document-time-drill-down branch July 17, 2026 17:34
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.

2 participants