[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#13563: tabulated-list-print-col nicer help-echo
From: |
Jonas Bernoulli |
Subject: |
bug#13563: tabulated-list-print-col nicer help-echo |
Date: |
Sat, 26 Jan 2013 22:14:37 +0100 |
User-agent: |
mu4e 0.9.9.5-dev6; emacs 24.3.50.1 |
`tabulated-list-set-col' adds a help-echo property to the "labels" that
it inserts. With this patch this is done a bit differently:
* If the label already has a help-echo property then use that help-echo
instead of overriding it with a generic one. This allows displaying
additional information when it makes sense.
* Otherwise propertize as before but don't prefix the help-echo with the
column name. That information is not really useful.
>From bbe78adda05897c892ea3c3687912a4c056d6fe1 Mon Sep 17 00:00:00 2001
From: Jonas Bernoulli <jonas@bernoul.li>
Date: Sat, 26 Jan 2013 22:03:09 +0100
Subject: [PATCH] lisp/emacs-lisp/tabulated-list.el (tabulated-list-print-col):
don't prepend help-echo with column name, if help-echo is already predefined
don't replace it
---
lisp/emacs-lisp/tabulated-list.el | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lisp/emacs-lisp/tabulated-list.el
b/lisp/emacs-lisp/tabulated-list.el
index 94b3c15..13fc730 100644
--- a/lisp/emacs-lisp/tabulated-list.el
+++ b/lisp/emacs-lisp/tabulated-list.el
@@ -363,7 +363,6 @@ Return the column number after insertion."
(right-align (plist-get props :right-align))
(label (if (stringp col-desc) col-desc (car col-desc)))
(label-width (string-width label))
- (help-echo (concat (car format) ": " label))
(opoint (point))
(not-last-col (< (1+ n) (length tabulated-list-format))))
;; Truncate labels if necessary (except last column).
@@ -379,7 +378,9 @@ Return the column number after insertion."
(setq width (- width shift))
(setq x (+ x shift))))
(if (stringp col-desc)
- (insert (propertize label 'help-echo help-echo))
+ (insert (if (get-text-property 0 'help-echo label)
+ label
+ (propertize label 'help-echo label)))
(apply 'insert-text-button label (cdr col-desc)))
(let ((next-x (+ x pad-right width)))
;; No need to append any spaces if this is the last column.
--
1.8.0.1
- bug#13563: tabulated-list-print-col nicer help-echo,
Jonas Bernoulli <=