bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Thuna]

[-- Attachment #1: Type: text/plain, Size: 283 bytes --]


Previously tabulted-list-widen-current-column failed to work when point
was at or ahead of a button or an image.  It now checks for image width
and button width.  Button width calculation doesn't account for the
properties of the button, however, so that is left there as a FIXME.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Fix-tabulated-list-widen-current-column-not-working-.patch --]
[-- Type: text/x-diff, Size: 1757 bytes --]

From c5d6b80867f7f02b8ecbc5ed7dc8009c570831ac Mon Sep 17 00:00:00 2001
From: Thuna <thuna.cing@gmail.com>
Date: Mon, 24 Jan 2022 12:40:13 +0300
Subject: [PATCH] Fix tabulated-list-widen-current-column not working for
 non-strings

* tabulated-list.el (tabulated-list-widen-current-column): Consider
buttons and images when calculating the width of the contents of a
cell.
---
 lisp/emacs-lisp/tabulated-list.el | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/lisp/emacs-lisp/tabulated-list.el b/lisp/emacs-lisp/tabulated-list.el
index 7ad4f7f863..2ace7ebb35 100644
--- a/lisp/emacs-lisp/tabulated-list.el
+++ b/lisp/emacs-lisp/tabulated-list.el
@@ -734,7 +734,16 @@ tabulated-list-widen-current-column
                        (max (setq col-width
                                   (cadr (aref tabulated-list-format
                                               col-nb)))
-                            (string-width (aref entry col-nb)))
+                            (let ((desc (aref entry col-nb)))
+                              (cond
+                               ((stringp desc)
+                                (string-width desc))
+                               ((eq (car desc) 'image)
+                                (car (image-size desc)))
+                               (t (string-width (car desc))
+                                  ;; FIXME: Take into consideration the properties
+                                  ;;        of the button when calculating width
+                                  ))))
                        (or (plist-get (nthcdr 3 (aref tabulated-list-format
                                                       col-nb))
                                       :pad-right)
-- 
2.25.1

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Lars Ingebrigtsen]

Thuna <thuna.cing@gmail.com> writes:

> Previously tabulted-list-widen-current-column failed to work when point
> was at or ahead of a button or an image.  It now checks for image width
> and button width.  Button width calculation doesn't account for the
> properties of the button, however, so that is left there as a FIXME.

I didn't know that tabulated-list-mode worked with images at all -- I'd
have thought you'd needed pixel calculations for that to work nicely?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Thuna]

> I didn't know that tabulated-list-mode worked with images at all -- I'd
> have thought you'd needed pixel calculations for that to work nicely?

It really depends on :align-to display property and that seems to handle
non-integer values properly.  It seems to work so long as the image
isn't wider than the column itself so the next thing to do is to patch
in image collapsing into print-col.

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Lars Ingebrigtsen]

Thuna <thuna.cing@gmail.com> writes:

> It really depends on :align-to display property and that seems to handle
> non-integer values properly.  It seems to work so long as the image
> isn't wider than the column itself so the next thing to do is to patch
> in image collapsing into print-col.

I think trying to expand tabulated-list-mode into elements that don't
have the same size isn't going to work well.  We're planning to add a
new library for columnar display of variable pitch stuff instead (so
that you can use images, mix fonts and variable pitch fonts).

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Thuna]

> I think trying to expand tabulated-list-mode into elements that don't
> have the same size isn't going to work well.  We're planning to add a
> new library for columnar display of variable pitch stuff instead (so
> that you can use images, mix fonts and variable pitch fonts).

I don't know if a different library is necessary for columnar display.
I believe tabulated-list-mode has the necessary foundations for such a
purpose.  I imagine with additions to the tabulated-list-format's
accepted props such as :content-type, desired features can be
implemented without the need to rewrite all the code.


PS: I have recently attempted to mail emacs-devel@gnu.org (to
help-gnu-emacs@gnu.org as well) regarding the wanted behavior of images
as I believe that is where this conversation belongs to however my mails
seem to not be getting through.  I am unsure why this the case as my
mails are seen as sent on my end.

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Lars Ingebrigtsen]

Thuna <thuna.cing@gmail.com> writes:

> I don't know if a different library is necessary for columnar display.
> I believe tabulated-list-mode has the necessary foundations for such a
> purpose.

I don't -- it would require a pretty full rewrite to handle these
things.  (And tabulated-list-mode has many other problems, like not
being able to have other text in the buffer, etc.)

So adding things like this, in my opinion, just makes
tabulated-list-mode more convoluted, but won't work properly for most
people, so I don't think we'll try to take tabulated-list-mode in this
direction, and I'm therefore closing this bug report.

> PS: I have recently attempted to mail emacs-devel@gnu.org (to
> help-gnu-emacs@gnu.org as well) regarding the wanted behavior of images
> as I believe that is where this conversation belongs to however my mails
> seem to not be getting through.  I am unsure why this the case as my
> mails are seen as sent on my end.

Seems like they're getting through OK to me?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#53498: [PATCH] Fix tabulated-list-widen-current-column not working for non-strings

[Thuna]

> So adding things like this, in my opinion, just makes
> tabulated-list-mode more convoluted, but won't work properly for most
> people, so I don't think we'll try to take tabulated-list-mode in this
> direction, and I'm therefore closing this bug report.

I understand the reasoning for not supporting images however this patch
fixes the bug for buttons as well, which I assume will be supported.