search

halfbrained / cuda_lsp

LSP Client plugin for CudaText
6 stars 4 forks source link

Handling server notifications/logs #76

Closed halfbrained closed 3 years ago

halfbrained commented 3 years ago

I think maybe give each server a separate panel for their outputs? LSP servers give logs, and messages to show in UI, both with severity (log, info, warning, error)

For now I'm putting logs to the Output panel, and messages to the Console, but it would be nice to highlight these properly according to severity; using 'NOTE: ' and '*Error: ' of the Console panel wouldn't look too nice.

Alexey-T commented 3 years ago

it is a good idea. maybe give in lsp_*.json the option "panel": true or even "panel": "error,warning,info,log" (show only mentioned kinds) ?

Alexey-T commented 3 years ago

(the implementaion of panel-- can be seen in FTP plg.)

halfbrained commented 3 years ago

Ok, panel will be 'on' by default, and disabled with "panel": false in server or plugin configs.

And severity filter will be in the panel's form or context menu - since user might want to toggle these frequently

Alexey-T commented 3 years ago

yes, sounds good.

halfbrained commented 3 years ago

Added (https://github.com/halfbrained/cuda_lsp/commit/f63b10b3f3d6a32000ae720edfa1542002c3cc8c)

log_panel

Alexey-T commented 3 years ago

Looks very good! I can publish it in a few days, not now