From: Dmitry Gutov <dgutov@yandex.ru>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 20847@debbugs.gnu.org
Subject: bug#20847: [display engine] 25.0.50; company-mode popup makes point jump to an entirely different location
Date: Sun, 21 Jun 2015 21:06:07 +0300 [thread overview]
Message-ID: <5586FD0F.8090300@yandex.ru> (raw)
In-Reply-To: <83h9q1hv0t.fsf@gnu.org>
On 06/21/2015 07:43 PM, Eli Zaretskii wrote:
> Any scenario where a screen line ends in a newline that comes from an
> overlay string. Try several such scenarios, and then tell me whether
> the place we display the cursor looks better than the alternative.
Hmm, yeah, after sounds better that before. But we do display point in
the margin on step 6. So it must be possible.
> Emacs cannot move cursor except by moving point, I'm sure you know
> that. The only exception is when we show the cursor on a display or
> overlay string, guided by the 'cursor' property. There are no other
> exceptions.
Not really. I only know that *I* can't move cursor by any other means.
I also vaguely recall someone (yourself?) stating that this limitation
could be lifted without too much work.
>> See the bottom of `company--replacement-string'. If `cursor' is applied
>> unconditionally, and if I change the arguments 0 and 1 to 1 and 2, on
>> step 6 the cursor is displayed at the beginning of the next line (so we
>> know the change has effect), but the second problem (after step 9) is
>> still present.
>
> AFAICT, this will put the 'cursor' property on a character that is
> after the leading newline of the overlay string, yes?
True.
> If so, that's
> not going to work: you need th 'cursor' property on some glyph that is
> displayed on the same line where the newline is.
Augh.
Like mentioned in the previous message, I don't see a good,
non-user-confusing place for it on the same line. The beginning of the
next line would've worked reasonably well, though.
I'd also accept the cursor not being displayed at all.
> That is, you need to
> make at least one character of "hel" part of the overlay string, and
> put the 'cursor' property on it, making its value large enough to
> "cover" the position of the newline.
I was kinda hoping that "overlay with display string starting with
newline" was the only issue.
>> - The bug only manifests after the step 9 (backspacing), whereas the
>> whole explanation seems to apply to the step 6 as well. Yet, point stays
>> in place there.
>
> Like I said, I didn't investigate that. I think some redisplay
> optimization is responsible. If it's important to have the same
> (mis)behavior in both cases, I can look into that.
Couldn't the same optimization have a reason to be enabled in both
cases? It might be worth investigating, at least.
Consistent behavior would also be good; so that the users don't have to
try too hard to catch problematic cases like this one.
>The bidirectional display doesn't have that
> luxury, so it needs additional support information, and that
> information is simply absent in this case.
I see.
next prev parent reply other threads:[~2015-06-21 18:06 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-19 1:04 bug#20847: [display engine] 25.0.50; company-mode popup makes point jump to an entirely different location Dmitry Gutov
2015-06-19 19:07 ` Eli Zaretskii
2015-06-20 11:51 ` Eli Zaretskii
2015-06-21 13:56 ` Dmitry Gutov
2015-06-21 16:43 ` Eli Zaretskii
2015-06-21 18:06 ` Dmitry Gutov [this message]
2015-06-21 18:24 ` Eli Zaretskii
2015-06-21 19:23 ` Eli Zaretskii
2015-06-21 20:17 ` Dmitry Gutov
2015-06-21 20:02 ` Dmitry Gutov
2015-06-22 2:45 ` Eli Zaretskii
2015-06-22 11:01 ` Dmitry Gutov
2015-06-22 13:40 ` Dmitry Gutov
2015-06-22 16:26 ` Eli Zaretskii
2015-06-22 21:06 ` Dmitry Gutov
2015-06-23 16:39 ` Eli Zaretskii
2015-06-23 18:44 ` Dmitry Gutov
2015-06-23 19:07 ` Eli Zaretskii
2015-06-23 21:15 ` Dmitry Gutov
2015-06-24 16:18 ` Eli Zaretskii
2015-06-29 15:48 ` Dmitry Gutov
2015-06-30 17:46 ` Eli Zaretskii
2015-06-30 19:41 ` Dmitry Gutov
2015-06-30 20:11 ` Eli Zaretskii
2015-06-30 20:20 ` Dmitry Gutov
2015-07-01 2:42 ` Eli Zaretskii
2015-07-01 10:21 ` Dmitry Gutov
2015-07-01 15:16 ` Eli Zaretskii
2015-07-01 16:36 ` Dmitry Gutov
2015-07-01 16:40 ` Eli Zaretskii
2015-06-21 13:30 ` Dmitry Gutov
2015-06-21 14:16 ` Dmitry Gutov
2015-06-21 16:24 ` Eli Zaretskii
2015-06-21 17:46 ` Dmitry Gutov
2015-06-21 18:09 ` Eli Zaretskii
2015-06-21 20:01 ` Dmitry Gutov
2015-06-22 2:43 ` Eli Zaretskii
2015-06-22 10:57 ` Dmitry Gutov
2015-06-22 16:23 ` Eli Zaretskii
2015-06-23 0:15 ` Dmitry Gutov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5586FD0F.8090300@yandex.ru \
--to=dgutov@yandex.ru \
--cc=20847@debbugs.gnu.org \
--cc=eliz@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.