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.

Sign up for GitHub

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

Jump to bottom

Make imenu hierarchical #145

Merged
merged 2 commits into from
Nov 7, 2018
Merged

Make imenu hierarchical #145

merged 2 commits into from
Nov 7, 2018

Conversation

mkcms
Copy link
Collaborator

@mkcms mkcms commented Nov 4, 2018

  • eglot.el (eglot-imenu): Use :containerName to build a nested imenu
    index alist.

@mkcms
Make imenu hierarchical
5965a22 
* eglot.el (eglot-imenu): Use :containerName to build a nested imenu
  index alist.
@mkcms mkcms requested a review from joaotavora November 4, 2018 15:40
@MaskRay
Copy link
Contributor

MaskRay commented Nov 4, 2018

textDocument/documentSymbol may return DocumentSymbol[] (hierarchical document symbol) which you might also want to take a look.

@mkcms
Copy link
Collaborator Author

mkcms commented Nov 5, 2018

textDocument/documentSymbol may return DocumentSymbol[] (hierarchical document symbol)

Thanks. Is there any server that does this?

@mkcms mkcms merged commit c664f40 into joaotavora:master Nov 7, 2018
@mkcms mkcms deleted the feature/hierarchical-imenu branch November 7, 2018 11:09
@MaskRay
Copy link
Contributor

MaskRay commented Nov 7, 2018 •
edited
Loading

ccls supports it.
Some typescript or javascript language server may support it.
There appears to be a pending clangd revision to support it.

See emacs-lsp/lsp-ui#179 emacs-lsp/lsp-mode#421

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

@joaotavora joaotavora Awaiting requested review from joaotavora

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mkcms @MaskRay