unofficial mirror of help-gnu-emacs@gnu.org
 help / color / mirror / Atom feed
* auto-complete popup is broken on master
@ 2018-06-25 15:43 John Shahid
  2018-06-25 15:49 ` John Shahid
  0 siblings, 1 reply; 3+ messages in thread
From: John Shahid @ 2018-06-25 15:43 UTC (permalink / raw)
  To: help-gnu-emacs@gnu.org




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

* Re: auto-complete popup is broken on master
  2018-06-25 15:43 auto-complete popup is broken on master John Shahid
@ 2018-06-25 15:49 ` John Shahid
  2018-06-25 16:45   ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: John Shahid @ 2018-06-25 15:49 UTC (permalink / raw)
  To: help-gnu-emacs@gnu.org


Sorry for the premature post.

The candidates in the auto-complete popup aren't aligned properly. This
seems to be a result of the change in the behavior of
`vertical-motion'. auto-complete tries to move vertically by getting the
column using `(car (posn-col-row (posn-at-point)))' then use the col
number as follows `(vertical-motion '(COL 1))'. This works as expected
on `4a7e74fea687011ee81dcbb02294bccd99b3a05f` but its behavior has
changed on master. Current master doesn't take into account
display-line-number width. Was that an intentional change ? If so, what
is the recommended way to get the previous behavior ?

While debugging this issue I also noticed that auto-complete popup-el
use overlay 'prefix property which isn't documented. How did that use to
work ? I could fix the prefix by using `line-prefix' instead of
`prefix'.

cheers,

-js



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

* Re: auto-complete popup is broken on master
  2018-06-25 15:49 ` John Shahid
@ 2018-06-25 16:45   ` Eli Zaretskii
  0 siblings, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2018-06-25 16:45 UTC (permalink / raw)
  To: help-gnu-emacs

> From: John Shahid <jvshahid@gmail.com>
> Date: Mon, 25 Jun 2018 15:49:06 +0000
> 
> The candidates in the auto-complete popup aren't aligned properly. This
> seems to be a result of the change in the behavior of
> `vertical-motion'. auto-complete tries to move vertically by getting the
> column using `(car (posn-col-row (posn-at-point)))' then use the col
> number as follows `(vertical-motion '(COL 1))'. This works as expected
> on `4a7e74fea687011ee81dcbb02294bccd99b3a05f` but its behavior has
> changed on master. Current master doesn't take into account
> display-line-number width. Was that an intentional change ? If so, what
> is the recommended way to get the previous behavior ?

Thanks.  This is not the right place for reporting bugs, certainly not
about the development codebase.  Please submit a bug report using
report-emacs-bug, and please include there a recipe to reproduce the
problem starting from "emacs -Q".

(I think what you describe is not a bug, but it's hard to tell without
all the details.  Also, the commit you mention is quite old.)

> While debugging this issue I also noticed that auto-complete popup-el
> use overlay 'prefix property which isn't documented.

Can you point at places in auto-complete which are using that
property?



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

end of thread, other threads:[~2018-06-25 16:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-25 15:43 auto-complete popup is broken on master John Shahid
2018-06-25 15:49 ` John Shahid
2018-06-25 16:45   ` Eli Zaretskii

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).