GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — fix(files): restore horizontal scroll in CSV and XLSX preview tables by waleedlatif1 · Pull Request #6550 · simstudioai/sim

Conversation

waleedlatif1 commented Aug 11, 2026

Copy link

Copy Markdown

Collaborator

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)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot] reacted with thumbs up emoji

All reactions

👍 1 reaction


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

c9e8174

#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 Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link

Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link

Copy Markdown

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.}

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link

Copy Markdown

Contributor

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}

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps Bot reviewed

Aug 11, 2026

View reviewed changes

Comment thread

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

Uh oh!

There was an error while loading. Please reload this page.

waleedlatif1 commented Aug 11, 2026

Copy link

Copy Markdown

Collaborator Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji

All reactions

👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

waleedlatif1 commented Aug 11, 2026

Copy link

Copy Markdown

Collaborator Author

@cursor review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor Bot reviewed

Aug 11, 2026

View reviewed changes

Comment thread

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

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

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

Uh oh!

There was an error while loading. Please reload this page.


          fix(files): scroll preview tables from one container, not two nested …

2068d9a

…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 commented Aug 11, 2026

Copy link

Copy Markdown

Collaborator Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji

All reactions

👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

waleedlatif1 commented Aug 11, 2026

Copy link

Copy Markdown

Collaborator Author

@cursor review

cursor[bot] reacted with thumbs up emoji

All reactions

👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor Bot reviewed

Aug 11, 2026

View reviewed changes

cursor Bot left a comment

Copy link

Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

✅ 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.}

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions


          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.

vercel Bot temporarily deployed to Preview

August 11, 2026 19:44

Inactive

waleedlatif1 commented Aug 11, 2026

Copy link

Copy Markdown

Collaborator Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji

All reactions

👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

waleedlatif1 merged commit bd91ab7 into staging

Aug 11, 2026

44 of 49 checks passed

Uh oh!

There was an error while loading. Please reload this page.

waleedlatif1 deleted the fix/csv-viewer-horizontal-scroll branch

August 11, 2026 20:06

This was referenced Aug 11, 2026

fix(files): let a mouse wheel scroll CSV and XLSX previews horizontally #6563

Merged

v0.8.0: desktop app, workflow UI, v2 endpoints #6606

Merged

This branch was previously deployed

1 inactive deployment

Preview — 50b75b45 Deployed Aug 11, 2026 by vercel[bot]

View deployment

Reviewers

greptile-apps[bot]

greptile-apps[bot] left review comments

cursor[bot]

cursor[bot] left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.

GeezerVison

Conversation

waleedlatif1 commented Aug 11, 2026

Summary

Type of Change

Testing

Checklist

Uh oh!

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

Uh oh!

Uh oh!

cursor Bot 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-apps Bot 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!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

waleedlatif1 commented Aug 11, 2026

Uh oh!

Uh oh!

This branch was previously deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

vercel Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

cursor Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.