GNU bug report logs - #66394
29.1; Make register-read-with-preview more useful

Previous Next

Package: emacs;

Reported by: Thierry Volpiatto <thievol <at> posteo.net>

Date: Sat, 7 Oct 2023 19:07:01 UTC

Severity: normal

Found in version 29.1

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eshel Yaron <me <at> eshelyaron.com>
To: Thierry Volpiatto <thievol <at> posteo.net>
Cc: michael_heerdegen <at> web.de, Eli Zaretskii <eliz <at> gnu.org>, stefankangas <at> gmail.com, monnier <at> iro.umontreal.ca, 66394 <at> debbugs.gnu.org
Subject: bug#66394: 29.1; Make register-read-with-preview more useful
Date: Mon, 04 Dec 2023 08:57:47 +0100
Hi Thierry,


Thierry Volpiatto <thievol <at> posteo.net> writes:

> Eshel Yaron <me <at> eshelyaron.com> writes:
>
>> The idea is to provide the nice of enhancements from Thierry's patch via
>> more minimal changes,
>
> Your patch fails to provide all the enhancements I had provided.

Thanks for testing.

Indeed, I only reimplemented the parts I saw as clearly beneficial.
Most importantly, my patch improves stuff without breaking other stuff.

Perhaps you can explain your use case for the rest of the changes, and
if there's a good and compatible way to add them I'll be happy to look
into it at some point.

> - No filtering.
> - No navigation.
> - No default registers.
> - No possibility to configure a new command added to register.

If you could elaborate about these bullets, and explain their use,
that'd be great.

> - You reintroduced the old implementation which was not wrote correctly
> about handling various keys, particularly C-g.

How so?  Works well over here, AFAICT, as it did in Emacs 29.

> - About overwriting, now you have to press "y" to confirm instead of
> pressing RET.

Yes, I think `y-or-n-p` is more user friendly here, don't you?


Cheers,

Eshel




This bug report was last modified 1 year and 210 days ago.

Previous Next


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