Skip to content

Conversation

@krishanbhasin-px
Copy link
Contributor

Description

When loading .csv files, the load in the text editor. As each section is often of different length, it becomes quite hard to quickly visually inspect how the rows/columns differ.

In this PR I propose rendering the CSV as a table, with an optional 'edit' button that takes the user back to the text-editor view.

Additional Information

Checklist

Before submitting your PR, please indicate which issues are either fixed or closed by this PR. See GitHub Help: Closing issues using keywords.

  • I am aware the project is currently in maintenance-only mode. See README
  • I am aware that translations MUST be made through Transifex and that this PR is NOT a translation update
  • I am making a PR against the master branch.
  • I am sure File Browser can be successfully built. See builds and development.

@krishanbhasin-px krishanbhasin-px marked this pull request as ready for review November 20, 2025 10:13
@krishanbhasin-px krishanbhasin-px requested a review from a team as a code owner November 20, 2025 10:13
@krishanbhasin-px krishanbhasin-px changed the title Render CSVs as table feat: render CSVs as table Nov 20, 2025
Copy link
Member

@hacdias hacdias left a comment

Choose a reason for hiding this comment

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

Thanks for this PR. I like the idea. My only comment goes to the top bar, which overlaps with the buttons in the preview. I'd rather have this a bit more clean and neatly fit into the interface.

Any ideas?

Image

@azdfzshffg
Copy link

UP! Great idea, @krishanbhasin-px we need this feature !

@hacdias hacdias merged commit 982405e into filebrowser:master Nov 29, 2025
6 checks passed
@jniggemann
Copy link

This is a great addition, thank you @krishanbhasin-px
Can we make the field separator configurable somehow? I guess comma is fine for US and a lot of other countries, but at least Europe mostly uses a semicolon. Plus some strange people use the pipe as a FS...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants