Thanks to whomever added the kind column to the symbol picker! #11338
-
I just upgraded from I took a quick skim to try and find the changes in the changelog https://github.com/helix-editor/helix/compare/9c479e6d..229784cc But couldn't find it (and I've barely touched Rust so didn't want to dive into the code too much) Again it's super helpful and looks great, thanks mystery contributor! |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 1 reply
-
Glad you enjoy it! That new column comes from #9647. Refactoring the picker as a table has been a goal for a while (since #2814 and #3053 at least) and builds on some lovely changes from @sudormrfbin. Also included in that change is some refactors to the way asynchronous actions like preview syntax highlighting and dynamic actions like workspace symbol search work and interactive global search (made possible by @pascalkuthe's excellent work in #8021). Symbol kind has been something we've wanted to add to the picker for a while. The original issue is in #5446 and we saw a few PRs for it based on the old Picker layouts - #2452 and #9774 at least (I think there are more) - but we wanted to figure out how to keep the UI clean (basically not concatenate the symbol kind into the old row text). Also do check out the new documentation for how to use columns in pickers at https://docs.helix-editor.com/master/pickers.html added by @rcorre in #11218 and try out the new global search that replaced |
Beta Was this translation helpful? Give feedback.
Glad you enjoy it! That new column comes from #9647. Refactoring the picker as a table has been a goal for a while (since #2814 and #3053 at least) and builds on some lovely changes from @sudormrfbin. Also included in that change is some refactors to the way asynchronous actions like preview syntax highlighting and dynamic actions like workspace symbol search work and interactive global search (made possible by @pascalkuthe's excellent work in #8021).
Symbol kind has been something we've wanted to add to the picker for a while. The original issue is in #5446 and we saw a few PRs for it based on the old Picker layouts - #2452 and #9774 at least (I think there are more) - but we wanted to f…