Skip to content

docs: document multi-cell selection and copy in table charts#11230

Merged
igorlukanin merged 7 commits into
masterfrom
igor/cub-2917-docs-multi-cell-copy
Jul 10, 2026
Merged

docs: document multi-cell selection and copy in table charts#11230
igorlukanin merged 7 commits into
masterfrom
igor/cub-2917-docs-multi-cell-copy

Conversation

@igorlukanin

Copy link
Copy Markdown
Member

Summary

Documents the new multi-cell selection and copy behavior in table charts: Shift+click / Shift+Arrow to select a rectangular range, then Cmd/Ctrl+C or the cell menu's Copy / Copy with headers to copy it as tab-separated values that paste into a spreadsheet as a grid.

Adds a "Selecting and copying multiple cells" section to the table chart page, next to the existing "Cell menu" docs.

@igorlukanin igorlukanin requested a review from keydunov as a code owner July 10, 2026 02:03
@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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


Code Review

Docs-only change adding a "Selecting and copying multiple cells" section to docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx. Reviewed against docs-mintlify/CLAUDE.md conventions.

Verdict: LGTM — no blocking issues (0 high, 0 medium, 1 nit)

Content

  • Placement (right after "Cell menu") is appropriate — the new section is a natural extension of the existing cell interaction docs.
  • Heading level (H2) matches surrounding sections; no stray H1.
  • Writing style is direct, uses second-person implicitly, keeps paragraphs short, and uses a bullet list — all consistent with the style guide.
  • Formatting (**bold** for UI affordances/keys, em dashes) matches the rest of the file.
  • No frontmatter/nav changes needed — this is an in-page addition.

Nits (non-blocking)

  • table.mdx:176 — "The range menu also offers Copy and Copy with headers" introduces the term "range menu" without prior definition. The existing "Cell menu" section above defines its counterpart explicitly ("Left-clicking a table cell opens a context menu…"). Consider a brief note on how the range menu appears (e.g. right-click on the selected range) so readers aren't left guessing. Not a blocker — could be tightened in a follow-up.

Nothing to flag on

  • Security / performance / testing: N/A (docs-only, no runtime surface).
  • No code or navigation (docs.json) changes required.
• Branch: `igor/cub-2917-docs-multi-cell-copy`

@mintlify

mintlify Bot commented Jul 10, 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 10, 2026, 2:06 AM

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

@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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


Code Review

Docs-only change: expands table.mdx with a "Selecting and copying cells" section covering both single-cell and range copy behavior. Reviewed against docs-mintlify/CLAUDE.md.

Verdict: LGTM — 0 blocking, 0 nits

Details
  • Read the diff and updated section (table.mdx:168-191)
  • Checked against docs-mintlify conventions (headings, style, components)
  • Verified no navigation (docs.json) changes are needed for an in-page addition

Content

  • Placement (right after "Cell menu") is natural — extends the existing cell-interaction docs.
  • Headings: H2 for the section, H3 for the two sub-cases; consistent with surrounding structure and no stray H1.
  • Style is direct, second-person, short paragraphs, bulleted steps — matches the style guide.
  • Formatting (**Cmd/Ctrl+C**, **Copy value**, etc.) matches the rest of the file.
  • The prior nit from the first review ("range menu" undefined) has been resolved — the copy now reads "the cell menu also offers Copy and Copy with headers", reusing the existing term and linking back to the cell menu section.
  • The tab/newline TSV behavior and the "raw (unformatted) value" clarification are called out clearly.

Nothing to flag

  • Security / performance / testing: N/A (docs-only, no runtime surface).
  • No code, frontmatter, or nav changes needed.
• Branch: `igor/cub-2917-docs-multi-cell-copy`

Comment on lines +175 to +176
- **Click** the cell to select it, then press **Cmd/Ctrl+C**, or
- Left-click the cell and choose **Copy value** from the [cell menu](#cell-menu).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

"click ... or left click ..." sounds weird. This could have been a single instruction instead of a two-item list.

@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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


Addressed review feedback

  • Merged the two-item "Copy a single cell" list into a single instruction

table.mdx:173-177 now reads as one sentence: "Click the cell to select it, then press Cmd/Ctrl+C or choose Copy value from the cell menu. Either way copies that cell's raw (unformatted) value." — removing the awkward "click … or left-click …" split.

Pushed as dfd8948 to igor/cub-2917-docs-multi-cell-copy.

@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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


Code Review

Docs-only update to docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx — the latest commit (0a5bb2e) adds click-and-drag as a third way to select a range. Reviewed against docs-mintlify/CLAUDE.md.

Verdict: LGTM — 0 blocking, 0 nits

Details
  • Read the current diff (table.mdx:168-191)
  • Checked against docs-mintlify conventions (headings, style, components)
  • Confirmed no docs.json or frontmatter changes needed for an in-page addition
  • Verified the prior review nit ("range menu" undefined) and the "click … or left-click …" phrasing feedback have both been resolved

Content

  • Placement (right after "Cell menu") remains natural — extends existing cell-interaction docs.
  • Heading levels (H2 section, H3 sub-cases) are consistent with the surrounding file; no stray H1.
  • Style is direct, second-person, short paragraphs, bulleted steps — matches the style guide.
  • Formatting (**Cmd/Ctrl+C**, **Copy value**, **Copy values with headers**, em dashes) matches the rest of the file.
  • The three range-selection methods (click-and-drag, Shift+click, Shift+Arrow) read cleanly as a single bullet with "or" separators.
  • The TSV behavior (tabs across columns, newlines across rows) and "raw (unformatted) value" clarification are called out clearly.
  • Internal link [cell menu](#cell-menu) reuses the existing anchor — good.

Nothing to flag

  • Security / performance / testing: N/A (docs-only, no runtime surface).
  • No code, frontmatter, or nav changes needed.

• Branch: igor/cub-2917-docs-multi-cell-copy

@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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


Code Review

Docs-only update to docs-mintlify/docs/explore-analyze/charts/chart-types/table.mdx. The latest commit (b030779) adds a note that the same select/copy behavior also works in the Results table on the query panel. Reviewed against docs-mintlify/CLAUDE.md.

Verdict: LGTM — 0 blocking, 0 nits

Details
  • Reviewed the current diff (table.mdx:168-194)
  • Checked against docs-mintlify conventions (headings, style, components)
  • Confirmed no docs.json or frontmatter changes needed for an in-page addition
  • Verified prior review feedback ("range menu" wording, "click … or left-click …" phrasing) remains resolved

Content

  • The new "The same single-cell and range selection works in the Results table on the query panel, not just the table visualization." sentence is a helpful callout — placed at the end of the section where it doesn't disrupt the primary flow, and phrased as an aside so readers focused on the table chart aren't distracted.
  • Heading structure (H2 section, H3 sub-cases) is consistent with the surrounding file; no stray H1.
  • Style is direct, second-person, short paragraphs, bulleted steps — matches the style guide.
  • Formatting (**Cmd/Ctrl+C**, **Copy value**, **Copy values**, **Copy values with headers**, **Results**) matches the rest of the file.
  • Internal link [cell menu](#cell-menu) reuses the existing anchor — good.
  • The three range-selection methods (click-and-drag, Shift+click, Shift+Arrow) still read cleanly as one bullet with "or" separators.
  • TSV behavior (tabs across columns, newlines across rows) and "raw (unformatted) value" clarification remain clear.

Nothing to flag

  • Security / performance / testing: N/A (docs-only, no runtime surface).
  • No code, frontmatter, or nav changes needed.

• Branch: igor/cub-2917-docs-multi-cell-copy

@claude

claude Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

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


Code Review

Latest commit (bb564eb) drops arrow-key selection, leaving click-and-drag + Shift+click as the two range-selection methods. Reviewed against docs-mintlify/CLAUDE.md.

Verdict: LGTM — 0 blocking, 0 nits

Details
  • Read the current section (table.mdx:168-192)
  • Checked against docs-mintlify conventions (headings, style, components)
  • Confirmed no docs.json or frontmatter changes needed
  • Verified prior review feedback ("range menu" wording, "click … or left-click …" phrasing) remains resolved

Content

  • Removing Shift+Arrow keeps the doc aligned with the actual scope of the feature; the remaining bullet ("Click and drag … or Shift+click …") reads cleanly.
  • Placement (right after "Cell menu"), heading levels (H2 section, H3 sub-cases), and style (direct, second-person, short paragraphs) all match the surrounding file and the style guide.
  • Formatting (**Cmd/Ctrl+C**, **Copy value**, **Copy values**, **Copy values with headers**, **Results**) is consistent.
  • TSV behavior (tabs across columns, newlines across rows) and the "raw (unformatted) value" clarification remain clear.
  • Internal link [cell menu](#cell-menu) reuses the existing anchor.
  • Results-table callout at the end remains useful as an aside.

Nothing to flag

  • Security / performance / testing: N/A (docs-only, no runtime surface).
  • No code, frontmatter, or nav changes needed.

• Branch: igor/cub-2917-docs-multi-cell-copy

@igorlukanin igorlukanin merged commit 10f8362 into master Jul 10, 2026
9 checks passed
@igorlukanin igorlukanin deleted the igor/cub-2917-docs-multi-cell-copy branch July 10, 2026 21: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