GNU bug report logs - #42039
27.0.91; (posn-x-y (posn-at-point)) inconsistent with display-line-numbers-mode

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 42039 in the body.
You can then email your comments to 42039 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Thu, 25 Jun 2020 16:29:21 +0300

1. Enable display-line-numbers-mode.
2. Move point to the beginning of some line.
3. Evaluate (posn-x-y (posn-at-point))

=> The value in CAR will be > 0 (to account for the columns taken by 
d-l-n-m).

4. Do the same thing on a line that belongs to an overlay. For example,
the overlay in the bug reporting buffer (move point to the line with
"This bug report...").
5. The return value will be like (0 . 180).

The expected behavior: the column should still account for the offset by
the d-l-n-m.

I'd really like to see this fixed in Emacs, it makes positioning of
popups unpredictable when display-line-numbers-mode is enabled: 
https://github.com/company-mode/company-quickhelp/issues/106

Message #8 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Thu, 25 Jun 2020 17:18:42 +0300

Sorry,

On 25.06.2020 16:29, Dmitry Gutov wrote:
> 
> I'd really like to see this fixed in Emacs, it makes positioning of

                                    ^ in Emacs 27

Message #11 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Thu, 25 Jun 2020 20:42:50 +0300

> From: Dmitry Gutov <dgutov <at> yandex.ru>
> Date: Thu, 25 Jun 2020 16:29:21 +0300
> 
> 1. Enable display-line-numbers-mode.
> 2. Move point to the beginning of some line.
> 3. Evaluate (posn-x-y (posn-at-point))
> 
> => The value in CAR will be > 0 (to account for the columns taken by 
> d-l-n-m).
> 
> 4. Do the same thing on a line that belongs to an overlay. For example,
> the overlay in the bug reporting buffer (move point to the line with
> "This bug report...").
> 5. The return value will be like (0 . 180).

The bug-report buffer doesn't use an overlay, it uses a display
property string.  Do you have an example with an overlay where this
problem happens, or is it limited to display strings?

Message #14 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Thu, 25 Jun 2020 21:35:59 +0300

On 25.06.2020 20:42, Eli Zaretskii wrote:
> The bug-report buffer doesn't use an overlay, it uses a display
> property string.  Do you have an example with an overlay where this
> problem happens, or is it limited to display strings?

Good question. It doesn't happen with just any overlay, so here's a full 
example:

1. Create an overlay, e.g.

(setq o (make-overlay (point) (1+ (point))))

The buffer contents seem unimportant. It can cover some text, or a 
newline, or multiple lines. But it should start at bol (if it doesn't 
start at bol, the result will also be unexpected, but not 0).

So the buffer text can just be a bunch of newlines, for the sake of this 
example.

2. Put a display string on it that (important!) ends with a newline:

(overlay-put o 'display "abc\ndef\n")

3. Eval

(car (posn-x-y (posn-at-point (overlay-start o))))

It will evaluate to 0 even if display-line-numbers-mode is enabled.

Message #17 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Thu, 25 Jun 2020 22:13:42 +0300

> Cc: 42039 <at> debbugs.gnu.org
> From: Dmitry Gutov <dgutov <at> yandex.ru>
> Date: Thu, 25 Jun 2020 21:35:59 +0300
> 
> 2. Put a display string on it that (important!) ends with a newline:
> 
> (overlay-put o 'display "abc\ndef\n")
> 
> 3. Eval
> 
> (car (posn-x-y (posn-at-point (overlay-start o))))
> 
> It will evaluate to 0 even if display-line-numbers-mode is enabled.

So this _is_ triggered only by display strings.  Thanks.

Message #20 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Fri, 26 Jun 2020 15:04:22 +0300

> From: Dmitry Gutov <dgutov <at> yandex.ru>
> Date: Thu, 25 Jun 2020 16:29:21 +0300
> 
> 1. Enable display-line-numbers-mode.
> 2. Move point to the beginning of some line.
> 3. Evaluate (posn-x-y (posn-at-point))
> 
> => The value in CAR will be > 0 (to account for the columns taken by 
> d-l-n-m).
> 
> 4. Do the same thing on a line that belongs to an overlay. For example,
> the overlay in the bug reporting buffer (move point to the line with
> "This bug report...").
> 5. The return value will be like (0 . 180).
> 
> The expected behavior: the column should still account for the offset by
> the d-l-n-m.
> 
> I'd really like to see this fixed in Emacs, it makes positioning of
> popups unpredictable when display-line-numbers-mode is enabled: 
> https://github.com/company-mode/company-quickhelp/issues/106

Your wish has been granted: this should now be fixed on the emacs-27
branch.

Message #23 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Fri, 26 Jun 2020 15:55:31 +0300

On 26.06.2020 15:04, Eli Zaretskii wrote:
>> From: Dmitry Gutov <dgutov <at> yandex.ru>
>> Date: Thu, 25 Jun 2020 16:29:21 +0300
>>
>> 1. Enable display-line-numbers-mode.
>> 2. Move point to the beginning of some line.
>> 3. Evaluate (posn-x-y (posn-at-point))
>>
>> => The value in CAR will be > 0 (to account for the columns taken by
>> d-l-n-m).
>>
>> 4. Do the same thing on a line that belongs to an overlay. For example,
>> the overlay in the bug reporting buffer (move point to the line with
>> "This bug report...").
>> 5. The return value will be like (0 . 180).
>>
>> The expected behavior: the column should still account for the offset by
>> the d-l-n-m.
>>
>> I'd really like to see this fixed in Emacs, it makes positioning of
>> popups unpredictable when display-line-numbers-mode is enabled:
>> https://github.com/company-mode/company-quickhelp/issues/106
> 
> Your wish has been granted: this should now be fixed on the emacs-27
> branch.

Thank you.

It's working fine now in the cases I really care about (so we can close 
the report), but here's one you seem to have missed:

1. M-x report-emacs-bug.
2. Toggle display-line-numbers-mode on.
3. Move point to the second display string ("If Emacs crashed...").
4. Evaluate (posn-x-y (posn-at-point)).

Message #26 received at 42039 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dgutov <at> yandex.ru>
Cc: 42039 <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Fri, 26 Jun 2020 16:37:50 +0300

> Cc: 42039 <at> debbugs.gnu.org
> From: Dmitry Gutov <dgutov <at> yandex.ru>
> Date: Fri, 26 Jun 2020 15:55:31 +0300
> 
> > Your wish has been granted: this should now be fixed on the emacs-27
> > branch.
> 
> Thank you.
> 
> It's working fine now in the cases I really care about (so we can close 
> the report), but here's one you seem to have missed:
> 
> 1. M-x report-emacs-bug.
> 2. Toggle display-line-numbers-mode on.
> 3. Move point to the second display string ("If Emacs crashed...").
> 4. Evaluate (posn-x-y (posn-at-point)).

It worked in a previous version of the fix, but then I decided to be
over-cautious...

Fixed.

Message #31 received at 42039-done <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 42039-done <at> debbugs.gnu.org
Subject: Re: bug#42039: 27.0.91; (posn-x-y (posn-at-point)) inconsistent with
 display-line-numbers-mode
Date: Fri, 26 Jun 2020 19:00:06 +0300

On 26.06.2020 16:37, Eli Zaretskii wrote:
> It worked in a previous version of the fix, but then I decided to be
> over-cautious...
> 
> Fixed.

Thanks!

Looking good now.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.