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

Document diagnostics customizations #195

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@iamFIREcracker
Copy link
Contributor

iamFIREcracker commented Oct 30, 2018

This PR mainly introduces two things:

  1. g:lsp_diagnostics_use_loclist option, so that users can use the location-list instead of the quickfix to display document diagnostics (I think it should be the default, but did not feel like introducing a breaking change)
  2. Does not open the loclist/quickfix when diagnostics information is received, unless the user has explicitly invoked LspDocumentDiagnostics

iamFIREcracker added some commits Oct 30, 2018

Add new g:lsp_diagnostics_use_loclist option
It defaults to 0, but when set to 1 it will force the plugin to use
the loclist instead of the quickfix to display document diagnostics
@prabirshrestha

This comment has been minimized.

Copy link
Owner

prabirshrestha commented Jan 2, 2019

@iamFIREcracker Can you sync with latest master and also change the contract to use array. This api would allows us in the future for anyone to register custom list ui so they can use CtrlP or FZF.

let g:lsp_list_ui = ['quickfix', 'loclist']
let g:lsp_diagnostics_list_ui = ['quickfix', 'loclist']

If g:lsp_diagnostics_list_ui is not set it needs to fallback to g:lsp_list_ui which defaults to ['quickfix', 'loclist'].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment