Skip to content

terminal: Fix hyperlinks not being detected correctly when preceded by box-drawing chars#48447

Merged
probably-neb merged 1 commit intozed-industries:mainfrom
Charlie-XIAO:issues/46795
Feb 13, 2026
Merged

terminal: Fix hyperlinks not being detected correctly when preceded by box-drawing chars#48447
probably-neb merged 1 commit intozed-industries:mainfrom
Charlie-XIAO:issues/46795

Conversation

@Charlie-XIAO
Copy link
Contributor

@Charlie-XIAO Charlie-XIAO commented Feb 5, 2026

Closes #46795

This PR aims to correctly detect the hyperlinks in cases like the following, where box-drawing characters immediately precede the path (without spaces in between).

╭─[Cargo.toml:55:1]
...
╰────

The only false negative with the fix in this PR would be that a file path really contains leading box drawing characters which I think is very unlikely.


Release Notes:

  • Fixed an issue where the hyperlinks would not be properly detected in the terminal if they were preceded by box-drawing characters (like ─, │, ┌, ┐)

…rawing chars

Signed-off-by: Charlie-XIAO <yx2436@nyu.edu>
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 5, 2026
@zed-community-bot zed-community-bot bot added the first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions label Feb 5, 2026
@MrSubidubi MrSubidubi changed the title terminal: fix hyperlink not detected correctly when preceded by box-drawing chars terminal: Fix hyperlinks not being detected correctly when preceded by box-drawing chars Feb 5, 2026
@probably-neb
Copy link
Collaborator

Makes sense that these shouldn't be expected to be part of a link. Nice work! Thanks!

@probably-neb probably-neb merged commit fc6f8d5 into zed-industries:main Feb 13, 2026
38 checks passed
@Charlie-XIAO Charlie-XIAO deleted the issues/46795 branch February 13, 2026 16:20
Caio-Ze pushed a commit to Caio-Ze/postprod-ide that referenced this pull request Feb 14, 2026
…y box-drawing chars (zed-industries#48447)

Closes zed-industries#46795

This PR aims to correctly detect the hyperlinks in cases like the
following, where box-drawing characters immediately precede the path
(without spaces in between).

```
╭─[Cargo.toml:55:1]
...
╰────
```

The only false negative with the fix in this PR would be that a file
path really contains leading box drawing characters which I think is
very unlikely.

---

Release Notes:

- Fixed an issue where the hyperlinks would not be properly detected in
the terminal if they were preceded by box-drawing characters (like ─, │,
┌, ┐)

Signed-off-by: Charlie-XIAO <yx2436@nyu.edu>
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 first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Terminal path_hyperlink_regexes doesn't match paths preceded by Unicode box-drawing characters

2 participants