fix(files): restore horizontal scroll in CSV and XLSX preview tables by waleedlatif1 · Pull Request #6550 · simstudioai/sim

waleedlatif1 · 2026-08-11T18:29:24Z

Summary

CSV/XLSX previews lost horizontal scroll and compressed every column into a vertical stack of letters
improvement(files): match CSV/XLSX preview tables to the markdown table chrome #6125 moved the previews onto the markdown table chrome, which gave both surfaces width: 100%. Right for prose, wrong for data — a CSV with dozens of columns divides the frame between them, and since that change also added overflow-wrap: anywhere, each column could break to a single character. The table never exceeded its frame, so overflow-x-auto had nothing to scroll
Split sizing out of the shared rule: prose tables keep width: 100%; preview tables get width: max-content; min-width: 100% and scroll
Bounded preview columns (min-width: 80px, max-width: 320px) so no column collapses to a sliver and one long value wraps instead of pushing the rest off-screen. 80px matches the tables grid's COL_WIDTH_MIN; 320px matches the capped content width used elsewhere in the app
Chrome (borders, padding, typography, header fill) stays shared — improvement(files): match CSV/XLSX preview tables to the markdown table chrome #6125's actual goal is preserved

Affects all three DataTable consumers: the CSV preview, the inline CSV path in preview-panel, and the XLSX preview.

Type of Change

Bug fix

Testing

Two tests added to document-table.test.ts, both confirmed to fail when the CSS is reverted. Full file-viewer suite passes (697 tests).

jsdom does no table layout, so the tests only assert the declarations resolve. Verified the actual layout separately in headless Chromium against a 16-column CSV:

	header row height	table vs frame	scrolls
before	137px	1202 / 1202	no
after	41px	2575 / 1217	yes
after, 3 columns	41px	1217 / 1217	no (fills frame)

Not yet exercised in the running app against a real uploaded CSV.

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)

#6125 moved the preview tables onto the markdown table chrome and gave both surfaces `width: 100%`. That is right for prose and wrong for data: a CSV with dozens of columns divides the frame between them, and since the same change added `overflow-wrap: anywhere`, every column was free to break down to a single character — so headers rendered as vertical columns of letters and the table never exceeded its frame, leaving `overflow-x-auto` with nothing to scroll. Split sizing out of the shared rule. Prose tables keep `width: 100%`; preview tables size to their content and scroll, with column bounds so no column collapses to a sliver and one long value wraps instead of pushing the rest off-screen. Chrome (borders, padding, typography, header fill) stays shared.

vercel · 2026-08-11T18:29:31Z

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 11, 2026 7:44pm

cursor · 2026-08-11T18:29:38Z

PR Summary

Low Risk
Scoped CSS and test updates in the file viewer preview path; no auth, data, or API changes.

Overview
Fixes CSV/XLSX preview tables that stopped scrolling horizontally and crushed wide sheets into single-character columns after shared markdown table styling applied width: 100% to both surfaces.

document-table.css now splits layout from shared chrome: markdown tables still use width: 100%, while .document-table previews use width: max-content and min-width: 100% so the table can grow past the viewer and overflow-x-auto on DataTable can scroll. Preview cells also get min-width: 80px and max-width: 320px so columns do not collapse or let one value dominate the row.

document-table.test.ts adds coverage that prose vs preview width rules differ and that preview column bounds resolve as expected.

^{Reviewed by Cursor Bugbot for commit c9e8174. Configure here.}

greptile-apps · 2026-08-11T18:31:53Z

Greptile Summary

The PR restores horizontal scrolling for CSV and XLSX previews while retaining shared visual styling with markdown tables.

Separates prose-table and preview-table sizing rules.
Sizes preview tables to their content while keeping narrow tables frame-width.
Bounds preview column widths and adds declaration-level regression tests.
Moves horizontal scrolling responsibility to each preview’s bounded parent container.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/data-table.tsx	Removes the nested horizontal scroll container so existing bounded preview containers own both-axis scrolling.
apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/document-table.css	Separates prose and preview sizing, using content-width preview tables and bounded columns without changing shared table chrome.
apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/document-table.test.ts	Adds regression assertions for the distinct table sizing and preview column bounds.

_{Reviews (4): Last reviewed commit: "fix(files): correct a stale reference to..." | Re-trigger Greptile}

waleedlatif1 · 2026-08-11T18:35:39Z

@greptile

waleedlatif1 · 2026-08-11T18:35:45Z

@cursor review

…ones DataTable owned `overflow-x-auto` while its caller owns the vertical scroll, so now that preview tables are actually wider than the frame the horizontal scrollbar rendered at the foot of the table rather than at the bottom of the viewport — up to 1,000 rows below it for the two callers whose container is a plain block (xlsx-preview, preview-panel). csv-table-preview escaped it only because its flex column compressed the wrapper to the frame height. Drop the inner overflow so the caller's bounded container scrolls both axes. All three callers now place the scrollbar at the viewport bottom.

waleedlatif1 · 2026-08-11T19:26:44Z

@greptile

waleedlatif1 · 2026-08-11T19:26:48Z

@cursor review

cursor

✅ 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 2068d9a. Configure here.}

The sizing comment still credited `.document-table`'s own `overflow-x-auto` for the horizontal scroll, which the previous commit removed in favour of the caller's container.

waleedlatif1 · 2026-08-11T19:59:15Z

@greptile

greptile-appsBot reviewed Aug 11, 2026
View reviewed changes

Comment threadapps/sim/app/workspace/[workspaceId]/files/components/file-viewer/document-table.css Outdated

cursorBot reviewed Aug 11, 2026
View reviewed changes

Comment threadapps/sim/app/workspace/[workspaceId]/files/components/file-viewer/document-table.css
Comment threadapps/sim/app/workspace/[workspaceId]/files/components/file-viewer/document-table.css

cursorBot reviewed Aug 11, 2026
View reviewed changes

fix(files): correct a stale reference to the removed inner overflow
50b75b4
The sizing comment still credited `.document-table`'s own `overflow-x-auto` for the horizontal scroll, which the previous commit removed in favour of the caller's container.

vercelBottemporarily deployed to Preview August 11, 2026 19:44 Inactive

waleedlatif1 merged commit bd91ab7 into stagingAug 11, 2026
44 of 49 checks passed

waleedlatif1 deleted the fix/csv-viewer-horizontal-scroll branch August 11, 2026 20:06

waleedlatif1 mentioned this pull request Aug 11, 2026
fix(files): let a mouse wheel scroll CSV and XLSX previews horizontally #6563
Merged
6 tasks

Navigation Menu

Uh oh!

fix(files): restore horizontal scroll in CSV and XLSX preview tables - #6550

fix(files): restore horizontal scroll in CSV and XLSX preview tables#6550
waleedlatif1 merged 3 commits into
stagingfrom
fix/csv-viewer-horizontal-scroll

waleedlatif1 commented Aug 11, 2026

Uh oh!

vercelBot commented Aug 11, 2026•
edited
Loading

Uh oh!

cursorBot commented Aug 11, 2026•
edited
Loading

Uh oh!

greptile-appsBot commented Aug 11, 2026•
edited
Loading

Important Files Changed

Uh oh!

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

Uh oh!

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

cursorBot left a comment

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

waleedlatif1 commented Aug 11, 2026

Summary

Type of Change

Testing

Checklist

Uh oh!

vercelBot commented Aug 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursorBot commented Aug 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

greptile-appsBot commented Aug 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Uh oh!

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

Uh oh!

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

vercelBot commented Aug 11, 2026•
edited
Loading

cursorBot commented Aug 11, 2026•
edited
Loading

greptile-appsBot commented Aug 11, 2026•
edited
Loading