Skip to content

Export filtered tables to CSV with coherent snapshots and failure-safe saving - #37

Merged
ProtonDev-sys merged 2 commits into
mainfrom
improve/table-export
Oct 12, 2026
Merged

ProtonDev-sys merged 2 commits into
mainfrom
improve/table-export

Conversation

@ProtonDev-sys

Copy link
Copy Markdown
Owner

Why

Users can copy one row, but cannot save the current filtered table for comparison or investigation. A Save dialog can also deliver background refresh messages, so an export must own its data before the dialog opens.

Changes

  • Add File > Export current table and Ctrl+Shift+S on populated table tabs.
  • Capture displayed rows, current sort/filter, complete cell text, and dragged visible-column order before showing the Save dialog. Category headings and hidden columns are omitted.
  • Write UTF-8 CSV with an encoding marker, quoted fields, escaped quotes, CRLF record endings, and text prefixes for formula-like values.
  • Save asynchronously in bounded chunks through an exclusively created temporary sibling; flush and close it before replacing the destination. Failure leaves the previous destination intact and cleans only the owned temporary.
  • Recover the action-busy state if the operating system cannot create an action worker.
  • Document snapshot semantics and display-value formatting.

Validation

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8b5b6bd4-721b-42d7-8d66-9f36af443083

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ProtonDev-sys
ProtonDev-sys merged commit a0a1f91 into main Oct 12, 2026
5 checks passed
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