Skip to content

agent_ui: Restore hover previews for image mentions - #58165

Merged
danilo-leal merged 1 commit into
zed-industries:mainfrom
cppcoffee:agent-ui-restore-hover-image
Jun 2, 2026
Merged

agent_ui: Restore hover previews for image mentions#58165
danilo-leal merged 1 commit into
zed-industries:mainfrom
cppcoffee:agent-ui-restore-hover-image

Conversation

@cppcoffee

@cppcoffee cppcoffee commented May 31, 2026

Copy link
Copy Markdown
Contributor

Self-Review Checklist:

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

Closes #58153

Release Notes:

  • Agent: Fixed image previews not being displayed after submitting a prompt and then hovering the mentioned image.

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 31, 2026
@dinocosta dinocosta added the area:ai/agent thread Feedback for Zed's Agent Thread label Jun 2, 2026

@danilo-leal danilo-leal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you so much!

@danilo-leal
danilo-leal enabled auto-merge June 2, 2026 23:22
@danilo-leal
danilo-leal added this pull request to the merge queue Jun 2, 2026
Merged via the queue into zed-industries:main with commit fb5deb8 Jun 2, 2026
45 checks passed
@cppcoffee
cppcoffee deleted the agent-ui-restore-hover-image branch June 3, 2026 01:56
dandv pushed a commit to dandv/zed that referenced this pull request Jun 3, 2026
…8165)

Self-Review Checklist:

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

Closes zed-industries#58153

Release Notes:

- Agent: Fixed image previews not being displayed after submitting a
prompt and then hovering the mentioned image.
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 8, 2026
…8165)

Self-Review Checklist:

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

Closes zed-industries#58153

Release Notes:

- Agent: Fixed image previews not being displayed after submitting a
prompt and then hovering the mentioned image.
dandv added a commit to dandv/zed that referenced this pull request Jun 9, 2026
Builds on the merged hover-preview work (zed-industries#58165). That change shows a
small thumbnail on hover, but large screenshots are unreadable at that
size. Clicking a pasted-image chip now materializes the image to a file
and opens it full-size in an editor tab.

Pasted images only exist as in-memory data, so the file is written under
the Zed data directory (not the OS temp dir) so it survives Zed restarts
and OS reboots, letting an open pasted-image tab be restored faithfully.
When the tab is closed, the editor entity is released and the file is
deleted - unless the app is quitting, in which case it is kept so the tab
can be restored on the next launch.

The handler is wired both for freshly pasted images and for images in
restored thread messages. File-attached images already open via their
on-disk path, so they are unaffected.

Also tidies the hover-preview decode helper from zed-industries#58165 to use the
codebase-standard `.log_err()` and return `Option<Arc<Image>>` instead of
a nested `Shared<Task<...>>`.

Release Notes:

- Improved pasted images in the agent panel: clicking a pasted-image chip now opens the image full-size in an editor tab.
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…8165)

Self-Review Checklist:

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

Closes zed-industries#58153

Release Notes:

- Agent: Fixed image previews not being displayed after submitting a
prompt and then hovering the mentioned image.
dandv added a commit to dandv/zed that referenced this pull request Jul 29, 2026
Builds on the merged hover-preview work (zed-industries#58165). That change shows a
small thumbnail on hover, but large screenshots are unreadable at that
size. Clicking a pasted-image chip now materializes the image to a file
and opens it full-size in an editor tab.

Pasted images only exist as in-memory data, so the file is written under
the Zed data directory (not the OS temp dir) so it survives Zed restarts
and OS reboots, letting an open pasted-image tab be restored faithfully.
When the tab is closed, the editor entity is released and the file is
deleted - unless the app is quitting, in which case it is kept so the tab
can be restored on the next launch.

The handler is wired both for freshly pasted images and for images in
restored thread messages. File-attached images already open via their
on-disk path, so they are unaffected.

Also tidies the hover-preview decode helper from zed-industries#58165 to use the
codebase-standard `.log_err()` and return `Option<Arc<Image>>` instead of
a nested `Shared<Task<...>>`.

Release Notes:

- Improved pasted images in the agent panel: clicking a pasted-image chip now opens the image full-size in an editor tab.
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…8165)

Self-Review Checklist:

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

Closes zed-industries#58153

Release Notes:

- Agent: Fixed image previews not being displayed after submitting a
prompt and then hovering the mentioned image.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ai/agent thread Feedback for Zed's Agent Thread cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Images are not shown in agent threads

3 participants