Skip to content

editor: Fix link navigation within editors that don't have a workspace#44389

Merged
benbrandt merged 1 commit intomainfrom
fix-channel-note-clicks
Dec 8, 2025
Merged

editor: Fix link navigation within editors that don't have a workspace#44389
benbrandt merged 1 commit intomainfrom
fix-channel-note-clicks

Conversation

@benbrandt
Copy link
Member

This mostly affects Channel Notes, but due to a change in #43921 we were short-circuiting before opening links.

I moved the workspace checks back to right before we need them so that we still follow the same control flow as usual for these editors.

Closes #44207

Release Notes:

  • N/A

This mostly affects Channel Notes, but due to a change in #43921 we were short-circuiting before opening links.

I moved the workspace checks back to right before we need them so that we still follow the same control flow as usual for these editors.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 8, 2025
@benbrandt benbrandt enabled auto-merge (squash) December 8, 2025 13:19
@benbrandt benbrandt merged commit 7ef4591 into main Dec 8, 2025
24 checks passed
@benbrandt benbrandt deleted the fix-channel-note-clicks branch December 8, 2025 13:36
CherryWorm pushed a commit to CherryWorm/zed that referenced this pull request Dec 16, 2025
zed-industries#44389)

This mostly affects Channel Notes, but due to a change in
zed-industries#43921 we were
short-circuiting before opening links.

I moved the workspace checks back to right before we need them so that
we still follow the same control flow as usual for these editors.

Closes zed-industries#44207

Release Notes:

- N/A
someone13574 pushed a commit to someone13574/zed that referenced this pull request Dec 16, 2025
zed-industries#44389)

This mostly affects Channel Notes, but due to a change in
zed-industries#43921 we were
short-circuiting before opening links.

I moved the workspace checks back to right before we need them so that
we still follow the same control flow as usual for these editors.

Closes zed-industries#44207

Release Notes:

- N/A
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.

Hyperlinks in channel notes files don't open when clicked

1 participant