Skip to content

csv_preview: #9 Allow copying cell and column header content - #61769

Merged
Anthony-Eid merged 4 commits into
zed-industries:mainfrom
HalavicH:feat/csv-preview/copy-content-buttons
Aug 13, 2026
Merged

csv_preview: #9 Allow copying cell and column header content#61769
Anthony-Eid merged 4 commits into
zed-industries:mainfrom
HalavicH:feat/csv-preview/copy-content-buttons

Conversation

@HalavicH

@HalavicH HalavicH commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Objective

  • There was no way to copy a cell's or a column header's text out of the CSV preview table (at all).

Solution

  • Right-clicking a table cell now copies its full content to the clipboard; the cell tooltip shows the content plus a "Right click to copy content" hint.
  • Right-clicking a column header name copies the column name to the clipboard, with the same tooltip pattern ("Right click to copy column name").

Testing

  • Right-click a cell: content is copied, tooltip shows the hint.
  • Right-click a column header: column name is copied, tooltip shows the hint.

Demo

image

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content adheres to Zed's UI standards (UX/UI and icon guidelines)
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Release Notes:

  • Added copy-to-clipboard on right-click for CSV preview cells and column headers

HalavicH added 4 commits July 28, 2026 10:17
Cell and header text had byte-for-byte identical right-click copy
handlers and tooltip builders; extracted into with_copy_on_right_click.
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jul 28, 2026
@HalavicH
HalavicH marked this pull request as ready for review July 28, 2026 10:30
@MrSubidubi MrSubidubi added the area:preview/csv Feedback for Zed's CSV support label Jul 28, 2026
@Anthony-Eid
Anthony-Eid added this pull request to the merge queue Aug 13, 2026
Merged via the queue into zed-industries:main with commit d866471 Aug 13, 2026
68 of 73 checks passed
@HalavicH HalavicH changed the title csv_preview: Allow copying cell and column header content csv_preview: #9 Allow copying cell and column header content Aug 14, 2026
playdohface pushed a commit to playdohface/zed that referenced this pull request Aug 29, 2026
…ries#61769)

# Objective

- There was no way to copy a cell's or a column header's text out of the
CSV preview table (at all).

## Solution

- Right-clicking a table cell now copies its full content to the
clipboard; the cell tooltip shows the content plus a "Right click to
copy content" hint.
- Right-clicking a column header name copies the column name to the
clipboard, with the same tooltip pattern ("Right click to copy column
name").

## Testing

- Right-click a cell: content is copied, tooltip shows the hint.
- Right-click a column header: column name is copied, tooltip shows the
hint.

## Demo

<img width="994" height="241" alt="image"
src="https://github.com/user-attachments/assets/827a0356-e152-4bfd-84ea-5745e638f13c"
/>


## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] Unsafe blocks (if any) have justifying comments
- [x] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Release Notes:

- Added copy-to-clipboard on right-click for CSV preview cells and
column headers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:preview/csv Feedback for Zed's CSV support cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants