Skip to content

OSC8-URIs support in terminal#54322

Merged
yara-blue merged 4 commits into
mainfrom
fix-OSC8-URIs
Apr 23, 2026
Merged

OSC8-URIs support in terminal#54322
yara-blue merged 4 commits into
mainfrom
fix-OSC8-URIs

Conversation

@yara-blue
Copy link
Copy Markdown
Member

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 #39339

Release Notes:

  • Fixed hyperlinks in the terminal on UNIX

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 20, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Apr 20, 2026
@yara-blue yara-blue requested a review from Veykril April 20, 2026 14:25
Comment thread crates/terminal/src/terminal_hyperlinks.rs Outdated
Comment thread crates/util/src/paths.rs
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.

I don't know about these changes. These functions are not just used for the terminal after all, so handing OSC 8 here doesn't feel right? Might be better to duplicate for osc 8 functionality instead

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ty you are right, makes no sense for "URLs" which is what this function is about.

I'll move it to terminal tomorrow and ping you 👍

@yara-blue yara-blue added this pull request to the merge queue Apr 23, 2026
Merged via the queue into main with commit 54d5eae Apr 23, 2026
31 checks passed
@yara-blue yara-blue deleted the fix-OSC8-URIs branch April 23, 2026 17:35
kathbigra pushed a commit to kathbigra/zed that referenced this pull request May 10, 2026
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#39339 

Release Notes:

- Fixed hyperlinks in the terminal on UNIX
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ctrl+Click on terminal file:// hyperlink works randomly

2 participants