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

LSP editor hovers are too intrusive (IEP-1308) #1032

Open
gtjoseph opened this issue Aug 19, 2024 · 1 comment
Open

LSP editor hovers are too intrusive (IEP-1308) #1032

gtjoseph opened this issue Aug 19, 2024 · 1 comment
Labels
configuration enhancement New feature or request

Comments

@gtjoseph
Copy link

Describe the bug
The LSP editor hovers are too intrusive and there seems to be no way to control them.

image
Do we really need a hover for a string literal??

This is the hover for a single structure member in the LSP editor...
image
Now here's the much better one from the CDT editor...
image

It's just too much.

To Reproduce
Steps to reproduce the behavior:

  1. Open any file in any example ESP_IDF example.
  2. Hover over things.

Expected behavior
A hover the same as the CDT editor

Screenshots
See above

Espressif-IDE Product Information:
prodinfo.txt

Eclipse Error log:
Window > Show View > Other > Search for "Error Log" (Attach as a file)

Please attach the error log as described here https://github.com/espressif/idf-eclipse-plugin#error-log

@github-actions github-actions bot changed the title LSP editor hovers are too intrusive LSP editor hovers are too intrusive (IEP-1308) Aug 19, 2024
@kolipakakondal kolipakakondal added configuration enhancement New feature or request labels Aug 20, 2024
@mmrein
Copy link

mmrein commented Nov 21, 2024

Gonna +1 this.

Also function hovers are not usefull at current state - first block is pretty much useless, second gives you whole lot of bloat from header file (if there is lot of comments), and the function prototype - the main and lot of times the only thing you need - is all the way down and you have to scroll for it.

BUT, before you can scroll, it acts like its loading something after you scroll first time, creating weird blink effect and really bad delay. So to see what you want you actually have to scroll TWICE.

Edit: Try esp_log_level_set() for example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configuration enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants