Skip to content

Commit e095b31

Browse files
niontrixTobias Heinlein
andauthored
[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 <niontrix@mailbox.org>
1 parent c3aa25d commit e095b31

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lsp-mode.el

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -524,7 +524,7 @@ supported by the language server."
524524
:group 'lsp-mode)
525525

526526
(defcustom lsp-eldoc-render-all nil
527-
"Display all of the info returned by document/onHover.
527+
"Display all of the info returned by textDocument/hover.
528528
If this is set to nil, `eldoc' will show only the symbol information."
529529
:type 'boolean
530530
:group 'lsp-mode)
@@ -5693,7 +5693,7 @@ When language is nil render as markup if `markdown-mode' is loaded."
56935693
(car (s-lines (s-trim (lsp--render-element contents)))))
56945694

56955695
(defun lsp--render-on-hover-content (contents render-all)
5696-
"Render the content received from `document/onHover' request.
5696+
"Render the content received from `textDocument/hover' request.
56975697
CONTENTS - MarkedString | MarkedString[] | MarkupContent
56985698
RENDER-ALL - nil if only the signature should be rendered."
56995699
(cond

0 commit comments

Comments
 (0)