Skip to content

Fix terminal path click failing when path is prefixed with '0:'#50663

Merged
SomeoneToIgnore merged 1 commit intozed-industries:mainfrom
xcb3d:fix/terminal-path-click-prefix-50531
Mar 4, 2026
Merged

Fix terminal path click failing when path is prefixed with '0:'#50663
SomeoneToIgnore merged 1 commit intozed-industries:mainfrom
xcb3d:fix/terminal-path-click-prefix-50531

Conversation

@xcb3d
Copy link
Copy Markdown
Contributor

@xcb3d xcb3d commented Mar 4, 2026

The path hyperlink regex's middle-char pattern [:(][^0-9()] allowed colon+space because space was not in the exclusion set. This caused 0: foo/bar.txt to be matched as a single path instead of just foo/bar.txt.

Fix: add space to the exclusion class: [:(][^0-9()\\ ]

Closes #50531

  • Added a solid test coverage and/or screenshots from doing manual testing
  • Done a self-review taking into account security and performance aspects
  • Aligned any UI changes with the UI checklist (N/A)

Release Notes:

…ndustries#50531)

The path hyperlink regex's middle-char pattern [:(][^0-9()] allowed
colon+space because space was not in the exclusion set. This caused
'0: foo/bar.txt' to be matched as a single path instead of just
'foo/bar.txt'.

Fix: add space to the exclusion class: [:(][^0-9()\\ ]
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 4, 2026
@maxdeviant maxdeviant changed the title fix: terminal path click fails when path is prefixed with '0:' Fix terminal path click failing when path is prefixed with '0:' Mar 4, 2026
@SomeoneToIgnore SomeoneToIgnore self-assigned this Mar 4, 2026
Copy link
Copy Markdown
Contributor

@SomeoneToIgnore SomeoneToIgnore left a comment

Choose a reason for hiding this comment

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

Looks good, thank you for the fix.

Just in case, I'll merge this after today's release to give this more time to be tested.

@SomeoneToIgnore SomeoneToIgnore merged commit 83b05f1 into zed-industries:main Mar 4, 2026
31 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Control-Clicking certain paths in the Terminal fails

2 participants