branch: elpa/popup
commit 5a892e53c65525363b7db725568dd0556a4f5f1f
Author: Takafumi Arakaki <aka....@gmail.com>
Commit: Takafumi Arakaki <aka....@gmail.com>

    Do not use summary-face for selected item
    
    The best way to do this is to dynamically create a
    face based on selection-face and summary-face.
    But this patch is still better than the older version
    without summary-face.
---
 popup.el | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/popup.el b/popup.el
index 9aae503..3caa82f 100644
--- a/popup.el
+++ b/popup.el
@@ -681,7 +681,7 @@ KEYMAP is a keymap that will be put on the popup contents."
         with popup-face = (popup-face popup)
         with mouse-face = (popup-mouse-face popup)
         with selection-face = (popup-selection-face popup)
-        with summary-face = (popup-summary-face popup)
+        with summary-face-0 = (popup-summary-face popup)
         with list = (popup-list popup)
         with length = (length list)
         with thum-size = (max (/ (* height height) (max length 1)) 1)
@@ -702,6 +702,7 @@ KEYMAP is a keymap that will be put on the popup contents."
         for face = (if (= i cursor)
                        (or (popup-item-selection-face item) selection-face)
                      (or (popup-item-face item) popup-face))
+        for summary-face = (unless (= i cursor) summary-face-0)
         for empty-char = (propertize " " 'face face)
         for scroll-bar-char = (if scroll-bar
                                   (cond

Reply via email to