unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
@ 2021-05-25  5:05 Jean Louis
  2021-09-19 17:07 ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Jean Louis @ 2021-05-25  5:05 UTC (permalink / raw)
  To: 48639


Tabulated list mode got quite a good support for zooming. Though I am
observing that if one of entries was too long that the next entry will
stick to the previous on decrease of text size. Video is to explain it:

https://gnu.support/files/tmp/2021-05-25/2021-05-25-08:00:37.ogv

In this case, on the video, the word "Task" is moved away from the
column because the previous entry is too long, the one which shows three
dots on the end.



In GNU Emacs 28.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.29, cairo version 1.17.4)
 of 2021-05-20 built on protected.rcdrun.com
Repository revision: 328efb47d04e3aa996bb8cd387d01c1a66ec29f5
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: Parabola GNU/Linux-libre

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
JSON LCMS2 LIBOTF LIBSYSTEMD LIBXML2 M17N_FLT MODULES NOTIFY INOTIFY
PDUMPER PNG RSVG SECCOMP SOUND THREADS TIFF TOOLKIT_SCROLL_BARS X11 XDBE
XIM XPM GTK3 ZLIB

Important settings:
  value of $LC_ALL: en_US.UTF-8
  value of $LANG: de_DE.UTF-8
  value of $XMODIFIERS: @im=exwm-xim
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message rmc puny dired dired-loaddefs
rfc822 mml mml-sec epa derived epg epg-config gnus-util rmail
rmail-loaddefs auth-source cl-seq eieio eieio-core cl-macs
eieio-loaddefs password-cache json map text-property-search time-date
subr-x seq byte-opt gv bytecomp byte-compile cconv mm-decode mm-bodies
mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader cl-loaddefs
cl-lib sendmail rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mail-utils
iso-transl tooltip eldoc electric uniquify ediff-hook vc-hooks
lisp-float-type mwheel term/x-win x-win term/common-win x-dnd tool-bar
dnd fontset image regexp-opt fringe tabulated-list replace newcomment
text-mode elisp-mode lisp-mode prog-mode register page tab-bar menu-bar
rfn-eshadow isearch easymenu timer select scroll-bar mouse jit-lock
font-lock syntax font-core term/tty-colors frame minibuffer cl-generic
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite charscript charprop
case-table epa-hook jka-cmpr-hook help simple abbrev obarray
cl-preloaded nadvice button loaddefs faces cus-face macroexp files
window text-properties overlay sha1 md5 base64 format env code-pages
mule custom widget hashtable-print-readable backquote threads dbusbind
inotify lcms2 dynamic-setting system-font-setting font-render-setting
cairo move-toolbar gtk x-toolkit x multi-tty make-network-process emacs)

Memory information:
((conses 16 53111 5760)
 (symbols 48 7127 3)
 (strings 32 19325 1817)
 (string-bytes 1 630436)
 (vectors 16 13240)
 (vector-slots 8 175057 10199)
 (floats 8 21 43)
 (intervals 56 200 0)
 (buffers 992 11))

-- 
Thanks,
Jean Louis

Take action in Free Software Foundation campaigns:
https://www.fsf.org/campaigns






^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-05-25  5:05 bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long Jean Louis
@ 2021-09-19 17:07 ` Stefan Kangas
  2021-09-19 17:37   ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2021-09-19 17:07 UTC (permalink / raw)
  To: Jean Louis; +Cc: 48639

tags 48639 + confirmed
thanks

Jean Louis <bugs@gnu.support> writes:

> Tabulated list mode got quite a good support for zooming. Though I am
> observing that if one of entries was too long that the next entry will
> stick to the previous on decrease of text size. Video is to explain it:
>
> https://gnu.support/files/tmp/2021-05-25/2021-05-25-08:00:37.ogv
>
> In this case, on the video, the word "Task" is moved away from the
> column because the previous entry is too long, the one which shows three
> dots on the end.

I can reproduce this on current master.

The problem can be observed in e.g. M-x list-processes, reducing the
width of a column with `{' until it is too narrow to fit the text (you
now see "..."), and finally `C-x -' to decrease the font size.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-09-19 17:07 ` Stefan Kangas
@ 2021-09-19 17:37   ` Eli Zaretskii
  2021-09-19 20:57     ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2021-09-19 17:37 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 48639, bugs

> From: Stefan Kangas <stefan@marxist.se>
> Date: Sun, 19 Sep 2021 10:07:39 -0700
> Cc: 48639@debbugs.gnu.org
> 
> > Tabulated list mode got quite a good support for zooming. Though I am
> > observing that if one of entries was too long that the next entry will
> > stick to the previous on decrease of text size. Video is to explain it:
> >
> > https://gnu.support/files/tmp/2021-05-25/2021-05-25-08:00:37.ogv
> >
> > In this case, on the video, the word "Task" is moved away from the
> > column because the previous entry is too long, the one which shows three
> > dots on the end.
> 
> I can reproduce this on current master.
> 
> The problem can be observed in e.g. M-x list-processes, reducing the
> width of a column with `{' until it is too narrow to fit the text (you
> now see "..."), and finally `C-x -' to decrease the font size.

I'm not sure I understand: is this about the fact that "C-x -" and
friends doesn't affect all the faces?





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-09-19 17:37   ` Eli Zaretskii
@ 2021-09-19 20:57     ` Stefan Kangas
  2021-09-20  4:31       ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2021-09-19 20:57 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 48639, bugs

Eli Zaretskii <eliz@gnu.org> writes:

>> The problem can be observed in e.g. M-x list-processes, reducing the
>> width of a column with `{' until it is too narrow to fit the text (you
>> now see "..."), and finally `C-x -' to decrease the font size.
>
> I'm not sure I understand: is this about the fact that "C-x -" and
> friends doesn't affect all the faces?

Hmm.  No, I don't think that's it.  Or maybe I don't understand what
you're saying.

What I see is that one column, immediately after one where text is
hidden with "04" due to lack of space, stops lining up.

So in `M-x list-processes' the first four columns look something like
this on my screen (the first line is the header line):

Process         PID       Status    Buffer
ielm            1651304 run          *ielm*
ispell          2423740 run          --
server          --        listen    --

Note how on the first and second line below the header, "run" does not
align with "Status" in the header like "listen" does.  Instead they end
up stuck next to "40".

This happens when you have first shrunk the column to be too small using
`{', and then scale the font size down using `C-x -'.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-09-19 20:57     ` Stefan Kangas
@ 2021-09-20  4:31       ` Eli Zaretskii
  2021-09-20  6:25         ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2021-09-20  4:31 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 48639, bugs

> From: Stefan Kangas <stefan@marxist.se>
> Date: Sun, 19 Sep 2021 13:57:53 -0700
> Cc: bugs@gnu.support, 48639@debbugs.gnu.org
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> The problem can be observed in e.g. M-x list-processes, reducing the
> >> width of a column with `{' until it is too narrow to fit the text (you
> >> now see "..."), and finally `C-x -' to decrease the font size.
> >
> > I'm not sure I understand: is this about the fact that "C-x -" and
> > friends doesn't affect all the faces?
> 
> Hmm.  No, I don't think that's it.  Or maybe I don't understand what
> you're saying.

I'm saying that some parts of the display are scaled by "C-x -" while
others aren't, and that is what causes misalignment.

> What I see is that one column, immediately after one where text is
> hidden with "04" due to lack of space, stops lining up.
> 
> So in `M-x list-processes' the first four columns look something like
> this on my screen (the first line is the header line):
> 
> Process         PID       Status    Buffer
> ielm            1651304 run          *ielm*
> ispell          2423740 run          --
> server          --        listen    --
> 
> Note how on the first and second line below the header, "run" does not
> align with "Status" in the header like "listen" does.  Instead they end
> up stuck next to "40".
> 
> This happens when you have first shrunk the column to be too small using
> `{', and then scale the font size down using `C-x -'.

list-processes is not a good example: it depends on what is going on
in the session, and in "emacs -Q" the display will generally be
empty.  Can you please use some other command, and explain there what
is wrong with scaling the display?  Because I don't think I understand
the descriptions so far, and the movie that was posted by the OP I
cannot watch here.

Thanks.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-09-20  4:31       ` Eli Zaretskii
@ 2021-09-20  6:25         ` Stefan Kangas
  2021-09-20  6:58           ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2021-09-20  6:25 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 48639, bugs

Eli Zaretskii <eliz@gnu.org> writes:

> I'm saying that some parts of the display are scaled by "C-x -" while
> others aren't, and that is what causes misalignment.

I could be wrong, but it seems to me that all fonts are scaling
correctly.  My guess is that it has to with how :align-to interacts with
selective display, either a bug in its interaction, or an incorrect use
of these features in 'tabulated-list-mode'.

> list-processes is not a good example: it depends on what is going on
> in the session, and in "emacs -Q" the display will generally be
> empty.  Can you please use some other command, and explain there what
> is wrong with scaling the display?

Try `list-packages'; move point to the "Name" column, hit `{' until you
see the "..." glyph indicating selective display, and finally `C-x -'.

Now, for some packages, but not all, the "Version" column no longer
has the correct horizontal alignment.  It is too far to the left.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-09-20  6:25         ` Stefan Kangas
@ 2021-09-20  6:58           ` Eli Zaretskii
  2021-09-24 22:46             ` Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2021-09-20  6:58 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 48639, bugs

> From: Stefan Kangas <stefan@marxist.se>
> Date: Sun, 19 Sep 2021 23:25:31 -0700
> Cc: bugs@gnu.support, 48639@debbugs.gnu.org
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > I'm saying that some parts of the display are scaled by "C-x -" while
> > others aren't, and that is what causes misalignment.
> 
> I could be wrong, but it seems to me that all fonts are scaling
> correctly.

The display is not made only of letters and numbers.

> My guess is that it has to with how :align-to interacts with
> selective display, either a bug in its interaction, or an incorrect use
> of these features in 'tabulated-list-mode'.
> 
> > list-processes is not a good example: it depends on what is going on
> > in the session, and in "emacs -Q" the display will generally be
> > empty.  Can you please use some other command, and explain there what
> > is wrong with scaling the display?
> 
> Try `list-packages'; move point to the "Name" column, hit `{' until you
> see the "..." glyph indicating selective display, and finally `C-x -'.
> 
> Now, for some packages, but not all, the "Version" column no longer
> has the correct horizontal alignment.  It is too far to the left.

That's because :align-to display spec is no longer there, isn't it?
there's only a single SPC character.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long
  2021-09-20  6:58           ` Eli Zaretskii
@ 2021-09-24 22:46             ` Stefan Kangas
  2023-09-06  1:57               ` bug#48639: bug#53133: 28.0.90; tabulated-list-sort doesn't work as expected Stefan Kangas
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Kangas @ 2021-09-24 22:46 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 48639, bugs

tags 48639 fixed
close 48639 28.1
thanks

Eli Zaretskii <eliz@gnu.org> writes:

>> Try `list-packages'; move point to the "Name" column, hit `{' until you
>> see the "..." glyph indicating selective display, and finally `C-x -'.
>>
>> Now, for some packages, but not all, the "Version" column no longer
>> has the correct horizontal alignment.  It is too far to the left.
>
> That's because :align-to display spec is no longer there, isn't it?
> there's only a single SPC character.

Indeed, that was the root cause of this.  This bug is now fixed on
master (commit 2767c89db7).





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#48639: bug#53133: 28.0.90; tabulated-list-sort doesn't work as expected
  2021-09-24 22:46             ` Stefan Kangas
@ 2023-09-06  1:57               ` Stefan Kangas
  0 siblings, 0 replies; 9+ messages in thread
From: Stefan Kangas @ 2023-09-06  1:57 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 53133, Eli Zaretskii, 48639-done, bugs

> Indeed, that was the root cause of this.  This bug is now fixed on
> master (commit 2767c89db7).

This bug was accidentally left open, so I'm closing it now.





^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2023-09-06  1:57 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-25  5:05 bug#48639: 28.0.50; tabulated-list-mode: columns move if previous entry was too long Jean Louis
2021-09-19 17:07 ` Stefan Kangas
2021-09-19 17:37   ` Eli Zaretskii
2021-09-19 20:57     ` Stefan Kangas
2021-09-20  4:31       ` Eli Zaretskii
2021-09-20  6:25         ` Stefan Kangas
2021-09-20  6:58           ` Eli Zaretskii
2021-09-24 22:46             ` Stefan Kangas
2023-09-06  1:57               ` bug#48639: bug#53133: 28.0.90; tabulated-list-sort doesn't work as expected Stefan Kangas

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).