From e095b31e2461d55ba60052a0f89950aec2c7251c Mon Sep 17 00:00:00 2001 From: Tobias Heinlein <46030637+niontrix@users.noreply.github.com> Date: Thu, 28 Nov 2024 10:13:59 +0100 Subject: [PATCH] [chore] Update comments with old `document/onHover` API (#4627) Why?: - In the current form of the LSP specification `document/onHover` is replaced by `textDocument/hover`. This change addresses the need by: - Replace `document/onHover` with `textDocument/hover` Co-authored-by: Tobias Heinlein --- lsp-mode.el | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lsp-mode.el b/lsp-mode.el index 5f81502ec9b..7ad8bcbc433 100644 --- a/lsp-mode.el +++ b/lsp-mode.el @@ -524,7 +524,7 @@ supported by the language server." :group 'lsp-mode) (defcustom lsp-eldoc-render-all nil - "Display all of the info returned by document/onHover. + "Display all of the info returned by textDocument/hover. If this is set to nil, `eldoc' will show only the symbol information." :type 'boolean :group 'lsp-mode) @@ -5693,7 +5693,7 @@ When language is nil render as markup if `markdown-mode' is loaded." (car (s-lines (s-trim (lsp--render-element contents))))) (defun lsp--render-on-hover-content (contents render-all) - "Render the content received from `document/onHover' request. + "Render the content received from `textDocument/hover' request. CONTENTS - MarkedString | MarkedString[] | MarkupContent RENDER-ALL - nil if only the signature should be rendered." (cond