[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
feature/icomplete-vertical 8a3ee9a 10/20: Fix icomplete prompt not visib
From: |
Jimmy Aguilar Mena |
Subject: |
feature/icomplete-vertical 8a3ee9a 10/20: Fix icomplete prompt not visible in icomplete-vertical (bug#24293) |
Date: |
Thu, 24 Sep 2020 22:58:20 -0400 (EDT) |
branch: feature/icomplete-vertical
commit 8a3ee9a0f85738e389f5d5d14255f763c5df6ef0
Author: Jimmy Aguilar Mena <spacibba@aol.com>
Commit: Jimmy Aguilar Mena <spacibba@aol.com>
Fix icomplete prompt not visible in icomplete-vertical (bug#24293)
* lisp/icomplete.el (icomplete--vertical-prospects) : Use pixels to
calculate the maximum number of visible prospects instead of line
numbers because different fonts may produce a wrong estimation.
---
lisp/icomplete.el | 34 ++++++++++++++++++----------------
1 file changed, 18 insertions(+), 16 deletions(-)
diff --git a/lisp/icomplete.el b/lisp/icomplete.el
index ccf85a2..ef46fd5 100644
--- a/lisp/icomplete.el
+++ b/lisp/icomplete.el
@@ -520,18 +520,21 @@ Conditions are:
(defun icomplete--vertical-prospects (match-braket prefix most _determ comps)
"List of vertical completions limited."
;; Max total rows to use, including the minibuffer content.
- (let ((prefix-len (and (stringp prefix)
- ;; Only hide the prefix if the corresponding info
- ;; is already displayed via `most'.
- (string-prefix-p prefix most t)
- (length prefix)))
- (prospects-rows (+ 1 (cl-count ?\n match-braket))) ;; prompt + row new
line around match
- (prospects-max-rows (cond ((floatp max-mini-window-height)
- (floor (* (frame-height)
max-mini-window-height)))
- ((integerp max-mini-window-height)
- max-mini-window-height)
- (t icomplete-prospects-height)))
- limit prospects comp)
+ (let* ((prefix-len (and (stringp prefix)
+ ;; Only hide the prefix if the corresponding info
+ ;; is already displayed via `most'.
+ (string-prefix-p prefix most t)
+ (length prefix)))
+ (line-height (line-pixel-height))
+ (prospects-max-height (cond ((floatp max-mini-window-height)
+ (floor (* max-mini-window-height
(frame-pixel-height))))
+ ((integerp max-mini-window-height)
+ (floor (* max-mini-window-height
line-height)))
+ (t
+ (* icomplete-prospects-height
line-height))))
+ ;; prompt + row new line around match
+ (prospects-rows-pixel (* (1+ (cl-count ?\n match-braket))
line-height))
+ limit prospects comp)
;; First candidate
(when (and prefix-len
@@ -539,16 +542,15 @@ Conditions are:
(push (substring (car comps) prefix-len) prospects)
(setq comps (cdr comps)
- prospects-rows (1+ prospects-rows)))
+ prospects-rows-pixel (+ prospects-rows-pixel line-height)))
;; The others
(while (and comps (not limit))
(setq comp (car comps)
comps (cdr comps))
+ (setq prospects-rows-pixel (+ prospects-rows-pixel line-height))
- (setq prospects-rows (1+ prospects-rows))
-
- (if (< prospects-rows prospects-max-rows)
+ (if (< prospects-rows-pixel prospects-max-height)
(push comp prospects)
(push icomplete-ellipsis prospects)
(setq limit t)))
- feature/icomplete-vertical 12b45f4 05/20: icomplete-vertical code simplify, (continued)
- feature/icomplete-vertical 12b45f4 05/20: icomplete-vertical code simplify, Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 98d65aa 06/20: Add modified local map for icomplete., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 2e757af 11/20: Improve vertical height calculation when not in minibuffer, Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical fd92ea8 09/20: Add special management of first candidate, Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 0c2cff3 14/20: convert icomplete-tidy-shadowed-file-names in a custom, Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 6194070 04/20: Icomplete-vertical fix first candidate and icomplete-hide-common-prefix., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical ca1d6ee 07/20: icomplete add item format., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 8bf2245 17/20: Fix error in formatting when completion was function %, Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 3ae232d 20/20: Icomplete improve vertical lines calculation., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 1743b757 19/20: icomplete--format-function process substring with prefix., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 8a3ee9a 10/20: Fix icomplete prompt not visible in icomplete-vertical (bug#24293),
Jimmy Aguilar Mena <=
- feature/icomplete-vertical d1b38d8 08/20: Add icomplete--format-function to format items., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 01c8197 16/20: Rename internal variables to follow convention., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 785d32a 18/20: Icomplete vertical respect icomplete-hide-common-prefix., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 91dabb1 01/20: Add variables for icomplete indicators., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 385c9d3 03/20: Improve icomplete-vertical implementation., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 9f6dfd8 12/20: Fix previous commit., Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 2ca463f 13/20: Improve formated size calculation, Jimmy Aguilar Mena, 2020/09/24
- feature/icomplete-vertical 2f1dcaa 15/20: Make some variable local or custom., Jimmy Aguilar Mena, 2020/09/24