emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[elpa] externals/eglot bd970be047: Fix #514: Reduce Eldoc noise from hov


From: ELPA Syncer
Subject: [elpa] externals/eglot bd970be047: Fix #514: Reduce Eldoc noise from hover messages
Date: Thu, 30 Jun 2022 03:57:34 -0400 (EDT)

branch: externals/eglot
commit bd970be047aba4513c2970b9ff2d842f0472cb0e
Author: jicksaw <jicksaw@pm.me>
Commit: GitHub <noreply@github.com>

    Fix #514: Reduce Eldoc noise from hover messages
    
    Also close #985
    
    Only echo hover response content, without response range.
    
    LSP specification says the range is meant to visualize a hover.
    Maybe echoing the range is useful for some, but it seems
    non-standard behavior.
    
    Example issue: haskell-language-server responds with range set to
    whole file when hovering a comment -> Large, useless eldoc
    
    * eglot.el (eglot--hover-info): Remove text selected by range from
    output
    
    Copyright-paperwork-exempt: Yes
---
 eglot.el | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/eglot.el b/eglot.el
index bde4a23f8e..b058183fb9 100644
--- a/eglot.el
+++ b/eglot.el
@@ -2711,13 +2711,10 @@ for which LSP on-type-formatting should be requested."
                (eglot--signal-textDocument/didChange)
                (eldoc)))))))))
 
-(defun eglot--hover-info (contents &optional range)
-  (let ((heading (and range (pcase-let ((`(,beg . ,end) (eglot--range-region 
range)))
-                              (concat (buffer-substring beg end)  ": "))))
-        (body (mapconcat #'eglot--format-markup
-                         (if (vectorp contents) contents (list contents)) 
"\n")))
-    (when (or heading (cl-plusp (length body))) (concat heading body))))
-
+(defun eglot--hover-info (contents &optional _range)
+  (mapconcat #'eglot--format-markup
+             (if (vectorp contents) contents (list contents)) "\n"))
+ 
 (defun eglot--sig-info (sigs active-sig sig-help-active-param)
   (cl-loop
    for (sig . moresigs) on (append sigs nil) for i from 0



reply via email to

[Prev in Thread] Current Thread [Next in Thread]