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

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

[elpa] externals/corfu c560320f43 3/3: corfu-popupinfo: Minor cleanup


From: ELPA Syncer
Subject: [elpa] externals/corfu c560320f43 3/3: corfu-popupinfo: Minor cleanup
Date: Sat, 19 Nov 2022 09:57:26 -0500 (EST)

branch: externals/corfu
commit c560320f430157097389baa45cd5251061b018e3
Author: Daniel Mendler <mail@daniel-mendler.de>
Commit: Daniel Mendler <mail@daniel-mendler.de>

    corfu-popupinfo: Minor cleanup
---
 extensions/corfu-popupinfo.el | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/extensions/corfu-popupinfo.el b/extensions/corfu-popupinfo.el
index c5e6c883dc..9c6b118308 100644
--- a/extensions/corfu-popupinfo.el
+++ b/extensions/corfu-popupinfo.el
@@ -211,15 +211,13 @@ The calculated area is in the form (X Y WIDTH HEIGHT DIR).
 DIR indicates the horizontal position direction of the info popup
 relative to the candidate popup, its value can be 'right or 'left."
   (pcase-let* ((border (alist-get 'child-frame-border-width 
corfu--frame-parameters))
-               ;; space between candidates popup and info popup
-               (space (- border))  ;; share the border
                (`(,_pfx ,_pfy ,pfw ,_pfh)
                 (corfu-popupinfo--frame-geometry (frame-parent corfu--frame)))
                (`(,cfx ,cfy ,cfw ,_cfh) (corfu-popupinfo--frame-geometry 
corfu--frame))
-               (x-on-right (+ cfx cfw space))
+               (x-on-right (+ cfx cfw (- border)))
+               (x-on-left (- cfx width border))
                (w-remaining-right (- pfw 1 x-on-right border border))
-               (x-on-left (- cfx space border width border))
-               (w-remaining-left (- cfx space 1 border border)))
+               (w-remaining-left (- cfx 1 border)))
     (cond
      ((>= w-remaining-right width)
       (list x-on-right cfy width height 'right))
@@ -239,7 +237,6 @@ The calculated area is in the form (X Y WIDTH HEIGHT DIR).
 DIR indicates the vertical position direction of the info popup
 relative to the candidate popup, its value can be 'bottom or 'top."
   (pcase-let* ((border (alist-get 'child-frame-border-width 
corfu--frame-parameters))
-               (space (- border))
                (lh (default-line-height))
                (`(,_pfx ,_pfy ,pfw ,pfh)
                 (corfu-popupinfo--frame-geometry (frame-parent corfu--frame)))
@@ -250,8 +247,8 @@ relative to the candidate popup, its value can be 'bottom 
or 'top."
                        (window-tab-line-height)
                        (or (cdr (posn-x-y (posn-at-point (point)))) 0)
                        lh)))
-               ;; (y-on-top (max 0 (- cfy space border height border)))
-               (y-on-bottom (+ cfy cfh space))
+               ;; (y-on-top (max 0 (- cfy height border)))
+               (y-on-bottom (+ cfy cfh (- border)))
                (h-remaining-top (- cfy border border))
                (h-remaining-bottom (- pfh y-on-bottom border border))
                (w-avail (min width (- pfw cfx border border))))
@@ -264,7 +261,7 @@ relative to the candidate popup, its value can be 'bottom 
or 'top."
       (setq height (min h-remaining-top height)
             height (min height (* (floor (/ height lh)) lh)))
       (list cfx
-            (max 0 (- cfy space border height border))
+            (max 0 (- cfy height border))
             w-avail height 'top))))
 
 (defun corfu-popupinfo--display-area (dir width height)



reply via email to

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