Skip to content

Add :lsp-forward-search command - #16380

Open
intertermux-code wants to merge 1 commit into
helix-editor:masterfrom
intertermux-code:fix/16310-lsp-forward-search
Open

intertermux-code wants to merge 1 commit into
helix-editor:masterfrom
intertermux-code:fix/16310-lsp-forward-search

Conversation

@intertermux-code

Copy link
Copy Markdown

%l was never expanded for forward search: no placeholder expansion exists in helix at all — texlab expands %l from the forwardSearch request position, which helix could never send (it fell back to line 0).

Fix: new typable :lsp-forward-search command sending textDocument/forwardSearch with the current cursor position to servers advertising the capability (standard client pattern, cf. texlab's VS Code extension). 4 files +165/-0.

cargo test -p helix-lsp: 13 passed (3 new); cargo check -p helix-term clean; fmt clean.

Fixes #16310.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Helix does not fill in the current line number at the placeholder %l

1 participant