Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for proposed folding provider. #169

Closed
angelozerr opened this issue Mar 15, 2018 · 11 comments
Closed

Support for proposed folding provider. #169

angelozerr opened this issue Mar 15, 2018 · 11 comments
Milestone

Comments

@angelozerr
Copy link

angelozerr commented Mar 15, 2018

VSCode provides folding provider for HTML, JSON.

VSCode has defined in their proposed https://github.com/Microsoft/vscode/blob/master/src/vs/vscode.proposed.d.ts#L37

See microsoft/language-server-protocol#418

It should be cool if lsp4j could provide this feature. Thanks!

@angelozerr
Copy link
Author

angelozerr commented Jul 26, 2018

Is there any chance that someone could work on this issue? This folding range is now included in the official spec https://github.com/Microsoft/vscode-languageserver-protocol-foldingprovider/blob/master/protocol.foldingProvider.md

@angelozerr
Copy link
Author

@spoenemann
Copy link
Contributor

Is there any chance that someone could work on this issue?

Maybe later this month. Contributions are welcome :-)

spoenemann added a commit that referenced this issue Aug 17, 2018
…ldingRanges (#169)

Signed-off-by: Miro Spönemann <miro.spoenemann@typefox.io>
@spoenemann spoenemann added this to the v0.5.0 milestone Aug 17, 2018
@angelozerr
Copy link
Author

angelozerr commented Aug 20, 2018

Thank a lot @spoenemann ! I will give you some feedback when I will implement folding support on client side (with LSP4e) and server side (with my XML Language Server lsp4xml)

Do you know when release of lsp4j will integrate folding support?

@spoenemann
Copy link
Contributor

v0.5.0 is on September 7th. We'll also have a milestone this week that includes it.

@angelozerr
Copy link
Author

Thanks @spoenemann !

@angelozerr
Copy link
Author

angelozerr commented Aug 20, 2018

@spoenemann there were a problem in the folding specification which was fixed at microsoft/language-server-protocol@09ecb5e

In other words, you need to rename the TextDocumentService#foldingRanges to remove the 's' of foldingRange like this:

CompletableFuture<List<FoldingRange>> foldingRange(FoldingRangeRequestParams params)

(foldingRanges to foldingRange)

@angelozerr
Copy link
Author

Thanks @spoenemann !

@angelozerr
Copy link
Author

@spoenemann just to give you feedback I have implemented folding on client side with LSP4E https://bugs.eclipse.org/bugs/show_bug.cgi?id=537706 and it works great (VSCode CSS Language Server folding is working).

For client side, I'm waiting for that 0.5.0 SNAPSHOT release will contain your fix with foldingRange to try it with my XML Language Server written in Java.

@spoenemann
Copy link
Contributor

The current state is already available in the 0.5.0.M1 milestone release.

@angelozerr
Copy link
Author

angelozerr commented Aug 23, 2018

The current state is already available in the 0.5.0.M1 milestone release.

0.5.0 SNAPSHOT is available today. I confirm you that my XML Language Server (server side) works well. Thanks again @spoenemann !

vladdu pushed a commit to vladdu/lsp4j that referenced this issue Jan 26, 2021
…ldingRanges (eclipse-lsp4j#169)

Signed-off-by: Miro Spönemann <miro.spoenemann@typefox.io>
vladdu pushed a commit to vladdu/lsp4j that referenced this issue Jan 26, 2021
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

No branches or pull requests

2 participants