Skip to content

feat(web): show imported CSV files as tables - #32

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-11873-show-imported-spreadsheets-as-tables
Nov 1, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-11873-show-imported-spreadsheets-as-tables

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

Part 1 of CL-11873: CSV only. Excel files follow in a separate PR.

What changed

  • @corbits/react-ui is bumped to 0.5.0 in the interface, the coder starter and the designer skill's reference (own commit).
  • A text material whose title ends in .csv renders with react-ui's interactive CsvTable: sort, selection and copy, spreadsheet numbering, name box and sums, column resize. Builder adds no grid of its own.
  • The pane bar gains "Download as CSV" and "Share as CSV"; clicking inside the grid no longer offers a quote.
  • A CSV stored encoded (not UTF-8) stays a plain file with a download link, as on main.

Verified

  • bun run lint, format:check, typecheck, test (70 pass).
  • Independent review; no code blockers.
  • Real UI on a throwaway stack: sort, name box, copy, Count/Average/Sum, the pane-bar download, dark mode and 390px. Screenshots below.

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Before and after

A 6-row CSV material, before (raw text) and after (table with download link).

BeforeAfter

@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11873

@corbits-triage corbits-triage 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.

Triage requests changes:

  • Conventions: added code matches a code pattern

@TheGreatAxios
TheGreatAxios marked this pull request as draft October 31, 2026 18:26
@TheGreatAxios
TheGreatAxios force-pushed the cl-11873-show-imported-spreadsheets-as-tables branch from 6972111 to 00c8846 Compare November 1, 2026 03:52
@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Screenshots

Real UI on a throwaway stack with a seeded sample products.csv (1280px unless noted).

The interactive grid:

Sorted, with a selection:

A range selection with Count, Average and Sum, and the pane bar's download:

Dark mode:

390px:

@TheGreatAxios
TheGreatAxios marked this pull request as ready for review November 1, 2026 17:33
@corbits-triage

corbits-triage Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Notes:

  • changes sensitive paths: locks: bun.lock, locks: workflows/coder/starter/bun.lock

Triage will approve, request review.

@corbits-triage
corbits-triage Bot dismissed their stale review November 1, 2026 17:34

Nothing triage asked to change still fails, so this request no longer stands.

corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 1, 2026

@corbits-triage corbits-triage 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.

Triage requests changes:

  • Conventions: added code matches a code pattern

@corbits-triage
corbits-triage Bot dismissed their stale review November 1, 2026 18:20

Nothing triage asked to change still fails, so this request no longer stands.

corbits-triage[bot]
corbits-triage Bot previously approved these changes Nov 1, 2026
An imported .csv material opens as a table with a download link instead
of raw text. A CSV stored encoded (not UTF-8) stays a download.
@TheGreatAxios
TheGreatAxios force-pushed the cl-11873-show-imported-spreadsheets-as-tables branch from 793b44d to a808221 Compare November 1, 2026 18:44
@TheGreatAxios
TheGreatAxios added this pull request to the merge queue Nov 1, 2026
Merged via the queue into main with commit 8941a9c Nov 1, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant