unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Aaron Jensen <aaronjensen@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 47860@debbugs.gnu.org, Gregory Heytings <gregory@heytings.org>
Subject: bug#47860: 28.0.50; Mini buffer resize when lines are truncated regression
Date: Mon, 19 Apr 2021 09:02:24 -0500	[thread overview]
Message-ID: <CAHyO48w7oNMQkd44JJH1y0X+W5VcZizwC4HRE74YSGGnFYxgeg@mail.gmail.com> (raw)
In-Reply-To: <83wnsys9ml.fsf@gnu.org>

 On Mon, Apr 19, 2021 at 8:10 AM Eli Zaretskii <eliz@gnu.org> wrote:
>
> > Date: Mon, 19 Apr 2021 12:40:12 +0000
> > From: Gregory Heytings <gregory@heytings.org>
> > cc: Eli Zaretskii <eliz@gnu.org>, 47860@debbugs.gnu.org
> >
> > Thanks.  But what do you expect this code to do?  I tested it, and for
> > Emacs 24 to 27 you see only "a" in the minibuffer.  After commit
> > 56c42bd28d, you see two lines, "a" and "bbb...".
>
> Exactly the questions to which I would like to know the answers,
> thanks.
>
> IOW, given that the current code does "somewhat" better than
> everything we had before, what exactly is the problem you (Aaron) see
> with the offending commit that you call it "regression"?

I have not tested with Emacs 27. I have, however, tested with Emacs 28
with and without the commit I mentioned.

With the commit reverted, I see this:

https://cln.sh/sNpBcb

Without it reverted, I see this:

https://cln.sh/RtPEie

What I expect is for the minibuffer to be sized as the first example
and not the second.

I have not tested my repro on Emacs 27, so it's interesting to hear
that only the first line is displayed. However,

This is what it looks like when using selectrum in Emacs 27:

https://user-images.githubusercontent.com/8199224/114367956-3f4a8e00-9b7d-11eb-8307-5372fb48de63.png

and Emacs 28:

https://user-images.githubusercontent.com/8588/114411541-e1fd0f80-9b71-11eb-8ba3-5bf1437a7806.png

In Emacs 28, the minibuffer is not resized to be large enough to see
all candidates. It also never scrolls, so it's difficult to pick any
candidate that is not visible.

I do know that Selectrum does some vertical resizing after adding text
to the minibuffer, so that may be what causes Emacs 27 to look right.
It may also be that that vertical resizing now fails in Emacs 28 for
some reason. I did not need the resizing code to reproduce what
appears to be *an* issue, but it could very well not be the exact
issue I'm seeing in Selectrum if it is indeed the resizing code that's
not working properly.

I'll spend a little time adding the resize code in and testing in
Emacs 27 to see if that helps narrow down to the exact issue I'm
seeing on selectrum.

Thanks,

Aaron





  reply	other threads:[~2021-04-19 14:02 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-18  1:31 bug#47860: 28.0.50; Mini buffer resize when lines are truncated regression Aaron Jensen
2021-04-18  6:57 ` Eli Zaretskii
2021-04-18 17:45   ` Aaron Jensen
2021-04-18 18:03     ` Eli Zaretskii
2021-04-19  0:15       ` Aaron Jensen
2021-04-19 12:40         ` Gregory Heytings
2021-04-19 13:10           ` Eli Zaretskii
2021-04-19 14:02             ` Aaron Jensen [this message]
2021-04-19 14:24               ` Aaron Jensen
2021-04-19 14:43                 ` Eli Zaretskii
2021-04-19 14:48               ` Eli Zaretskii
2021-04-19 15:13                 ` Aaron Jensen
2021-04-19 16:19                   ` Aaron Jensen
2021-04-19 17:15                     ` Eli Zaretskii
2021-04-19 17:40                       ` Eli Zaretskii
2021-04-22 11:07                         ` Eli Zaretskii
2021-04-22 16:00                           ` Aaron Jensen
2021-04-22 16:50                             ` Eli Zaretskii
2021-04-19 14:48               ` Gregory Heytings

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAHyO48w7oNMQkd44JJH1y0X+W5VcZizwC4HRE74YSGGnFYxgeg@mail.gmail.com \
    --to=aaronjensen@gmail.com \
    --cc=47860@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=gregory@heytings.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 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).