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

Un-flatten LSP symbol picker with a tree view. #5048

Closed
weakphish opened this issue Dec 7, 2022 · 1 comment
Closed

Un-flatten LSP symbol picker with a tree view. #5048

weakphish opened this issue Dec 7, 2022 · 1 comment
Labels
A-helix-term Area: Helix term improvements

Comments

@weakphish
Copy link
Contributor

weakphish commented Dec 7, 2022

LSP supports hierarchical symbols, but helix currently flattens then into a list and shows them with space+s. We could reuse the tree view that will soon be merged with the file tree (#2377) to display the symbol information in a tree interface.

Another option is to use tree sitter tags instead of LSP (not sure if it can report hierarchical information).

Originally posted by @sudormrfbin in #2425 (comment)

Another idea that I would like to explore is adding NerdFont symbols optionally to add a bit of visual flavor. This would be opt-in through an explicit TOML configuration.

@kirawi kirawi added the A-helix-term Area: Helix term improvements label Dec 9, 2022
@pascalkuthe
Copy link
Member

Discussed with @the-mikedavis . A tree view doesn't work well with the picker because fuzzy matching always resortd the list (and isn't super useful when it doesn't). In general tree doesn't really fit in with the picker.

Thisbkind of feature would require a dedicated ui like discussed in #2425 . Closing in favor of that issue

@pascalkuthe pascalkuthe closed this as not planned Won't fix, can't repro, duplicate, stale Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements
Projects
None yet
Development

No branches or pull requests

3 participants