search

phil294 / coffeesense

IntelliSense for CoffeeScript. LSP implementation / VSCode extension
MIT License
47 stars 8 forks source link

outline broken in vscode #7

Closed robert-boulanger closed 11 months ago

robert-boulanger commented 2 years ago

It seems the outline view in vscode is broken in combination with coffeesense See the attached picture. It show a class with several member functions which all are nested in each other inside the outline view. Also go to editor symbols (Shift-Cmd-P @) or Cmd-F12 does not work as expected. Symbos are shown, but any symbol inside the class body can not be targeted Bildschirmfoto 2022-01-29 um 19 14 29 .

phil294 commented 2 years ago

Yes, they are not really useful. Implementing proper symbols is not quite easy and bug-prone I suppose, but I haven't invested too much time into it either. Outline is based on symbols, and readme says about these:

Document symbols: Usable but not great. Check out Yorkxin's extension, it provides much better symbols if you need that

and I'd stick to that. Both extensions should get along just fine. It's a bit of waste of processor resources, but I wouldn't worry about that too much, as CoffeeSense is highly inefficient on its own already, due to the way it works internally.

phil294 commented 1 year ago

fyi in some of the recent releases, I have completely removed outline (document symbols) in order for the new sticky scroll feature of VSCode to nicely work, based on indentation.

As it stands, for this feature, I would still recommend the above linked extension by Yorkxin. It's now even deprecated (kindly in favor of CoffeeSense), but one can still install it.