search

emacs-lsp / lsp-ui

UI integrations for lsp-mode
https://emacs-lsp.github.io/lsp-ui
GNU General Public License v3.0
1.04k stars 139 forks source link

lsp-ui-imenu broken #527

Closed rileyrg closed 2 years ago

rileyrg commented 3 years ago

using the repo prepared for issue Cant set break point in top level js code of react app. lsp-ui-imenu is broken. Here handleClick is in the wrong place. Latest versions with emacs 27 from source.

Actual source if this is enough to reproduce:-

index.js

Screenshot from 2020-10-23 16-18-26 i

minikN commented 3 years ago

I think I experience the same image

jcs090218 commented 3 years ago

Is this issue related to #511? Or just somehow the imenu is broken when you startup? 😕

minikN commented 3 years ago

Is this issue related to #511? Or just somehow the imenu is broken when you startup?

No it's been like that before I started fiddling with auto refresh.

jcs090218 commented 3 years ago

No it's been like that before I started fiddling with auto refresh.

Got it! Thanks for the clarification! 👍

czikus commented 3 years ago

I'm seeing a similar problem in org-mode

brotzeit commented 2 years ago

Should be fixed.