Fix `lsp-ui-imenu` sub-title ordering.

emacs-lsp / lsp-ui

UI integrations for lsp-mode

https://emacs-lsp.github.io/lsp-ui

GNU General Public License v3.0

1.03k stars 141 forks source link

Fix `lsp-ui-imenu` sub-title ordering. #721

Closed DivvyCr closed 2 years ago

DivvyCr commented 2 years ago

The lsp-ui-imenu displayed items in the wrong hierarchy, which was previously found in Issues #474, #527 #632, and #714, and in Java from my experience:

I believe I found a quick fix, which arose due to multiple elisp overlays being used in a recursive function.

In any case, this is the result after my quick fix:

(Also, the multi-line change in the pull-request is just me removing a redundant let block)

brotzeit commented 2 years ago

I've tried it and it seems to work. Let's give it a try, thanks!