Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Michael Albinus]

eliz@gnu.org (Eli Zaretskii) writes:

Hi,

> diff --git a/etc/NEWS b/etc/NEWS
> index b7c4346..a847a88 100644
> --- a/etc/NEWS
> +++ b/etc/NEWS
>  +++
>  ** New transient mode 'repeat-mode' to allow shorter key sequences.
> +Type "M-x repeat-mode RET" to enable this mode.  You can then type
> +'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
> +instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
> +resize the selected window interactively, 'M-g n n p p' to navigate
> +next-error matches.  Any other key exits transient mode and then is

I'm not sure. Shouldn't this be rather "... exits repeat mode ..."?

Best regards, Michael.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Eli Zaretskii]

> From: Michael Albinus <michael.albinus@gmx.de>
> Cc: Eli Zaretskii <eliz@gnu.org>
> Date: Tue, 19 Oct 2021 15:09:11 +0200
> 
> eliz@gnu.org (Eli Zaretskii) writes:
> 
> Hi,
> 
> > diff --git a/etc/NEWS b/etc/NEWS
> > index b7c4346..a847a88 100644
> > --- a/etc/NEWS
> > +++ b/etc/NEWS
> >  +++
> >  ** New transient mode 'repeat-mode' to allow shorter key sequences.
> > +Type "M-x repeat-mode RET" to enable this mode.  You can then type
> > +'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
> > +instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
> > +resize the selected window interactively, 'M-g n n p p' to navigate
> > +next-error matches.  Any other key exits transient mode and then is
> 
> I'm not sure. Shouldn't this be rather "... exits repeat mode ..."?

Maybe.  But I didn't write that text, I only added the first sentence,
and refilled the rest.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Michael Albinus]

Eli Zaretskii <eliz@gnu.org> writes:

Hi,

>> > diff --git a/etc/NEWS b/etc/NEWS
>> > index b7c4346..a847a88 100644
>> > --- a/etc/NEWS
>> > +++ b/etc/NEWS
>> >  +++
>> >  ** New transient mode 'repeat-mode' to allow shorter key sequences.
>> > +Type "M-x repeat-mode RET" to enable this mode.  You can then type
>> > +'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
>> > +instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
>> > +resize the selected window interactively, 'M-g n n p p' to navigate
>> > +next-error matches.  Any other key exits transient mode and then is
>>
>> I'm not sure. Shouldn't this be rather "... exits repeat mode ..."?
>
> Maybe.  But I didn't write that text, I only added the first sentence,
> and refilled the rest.

I Know. But since you don't object, I believe I'm not wrong. Changed.

Best regards, Michael.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Juri Linkov]

>>> > diff --git a/etc/NEWS b/etc/NEWS
>>> > index b7c4346..a847a88 100644
>>> > --- a/etc/NEWS
>>> > +++ b/etc/NEWS
>>> >  +++
>>> >  ** New transient mode 'repeat-mode' to allow shorter key sequences.
>>> > +Type "M-x repeat-mode RET" to enable this mode.  You can then type
>>> > +'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
>>> > +instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
>>> > +resize the selected window interactively, 'M-g n n p p' to navigate
>>> > +next-error matches.  Any other key exits transient mode and then is
>>>
>>> I'm not sure. Shouldn't this be rather "... exits repeat mode ..."?
>>
>> Maybe.  But I didn't write that text, I only added the first sentence,
>> and refilled the rest.
>
> I Know. But since you don't object, I believe I'm not wrong. Changed.

This is more wrong than before.  'repeat-mode' can be disabled only
explicitly by the user.  Whereas any other key exits transient mode
enabled by 'set-transient-map'.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Michael Albinus]

Juri Linkov <juri@linkov.net> writes:

Hi Juri,

>>>> > diff --git a/etc/NEWS b/etc/NEWS
>>>> > index b7c4346..a847a88 100644
>>>> > --- a/etc/NEWS
>>>> > +++ b/etc/NEWS
>>>> >  +++
>>>> >  ** New transient mode 'repeat-mode' to allow shorter key sequences.
>>>> > +Type "M-x repeat-mode RET" to enable this mode.  You can then type
>>>> > +'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
>>>> > +instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
>>>> > +resize the selected window interactively, 'M-g n n p p' to navigate
>>>> > +next-error matches.  Any other key exits transient mode and then is
>>>>
>>>> I'm not sure. Shouldn't this be rather "... exits repeat mode ..."?
>>>
>>> Maybe.  But I didn't write that text, I only added the first sentence,
>>> and refilled the rest.
>>
>> I Know. But since you don't object, I believe I'm not wrong. Changed.
>
> This is more wrong than before.  'repeat-mode' can be disabled only
> explicitly by the user.  Whereas any other key exits transient mode
> enabled by 'set-transient-map'.

I see. But even in its original wording, it confuses me:

--8<---------------cut here---------------start------------->8---
Type 'M-x repeat-mode RET' to enable this mode.  You can then type
'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
resize the selected window interactively, 'M-g n n p p' to navigate
next-error matches.  Any other key exits transient mode and then is
executed normally.
--8<---------------cut here---------------end--------------->8---

At first glance, I don't know what is "executed normally". The command?
The key? But a key isn't "executed" I believe, it is rather handled.

And maybe it is also better to speak about "transient input mode", like
in other places in NEWS? This would give a better idea what it is about
to exit.

(I'm not much familiar with "transient modes", and so will be most of
the readers of this NEWS file)

Best regards, Michael.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Juri Linkov]

>> This is more wrong than before.  'repeat-mode' can be disabled only
>> explicitly by the user.  Whereas any other key exits transient mode
>> enabled by 'set-transient-map'.
>
> I see. But even in its original wording, it confuses me:
>
>   Type 'M-x repeat-mode RET' to enable this mode.  You can then type
>   'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
>   instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
>   resize the selected window interactively, 'M-g n n p p' to navigate
>   next-error matches.  Any other key exits transient mode and then is
>   executed normally.
>
> At first glance, I don't know what is "executed normally". The command?
> The key? But a key isn't "executed" I believe, it is rather handled.

Or more precisely: "The default key binding is used".

> And maybe it is also better to speak about "transient input mode", like
> in other places in NEWS? This would give a better idea what it is about
> to exit.

A transient input method is something completely different.

> (I'm not much familiar with "transient modes", and so will be most of
> the readers of this NEWS file)

From (info "(elisp) Controlling Active Maps"):

 -- Function: set-transient-map keymap &optional keep-pred on-exit
     This function adds KEYMAP as a “transient” keymap, which takes
     precedence over other keymaps for one (or more) subsequent keys.

So maybe "transient mode" should be replaced with "transient keymap"?

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Michael Albinus]

Juri Linkov <juri@linkov.net> writes:

Hi Juri,

>>> This is more wrong than before.  'repeat-mode' can be disabled only
>>> explicitly by the user.  Whereas any other key exits transient mode
>>> enabled by 'set-transient-map'.
>>
>> I see. But even in its original wording, it confuses me:
>>
>>   Type 'M-x repeat-mode RET' to enable this mode.  You can then type
>>   'C-x u u' instead of 'C-x u C-x u' to undo many changes, 'C-x o o'
>>   instead of 'C-x o C-x o' to switch windows, 'C-x { { } } ^ ^ v v' to
>>   resize the selected window interactively, 'M-g n n p p' to navigate
>>   next-error matches.  Any other key exits transient mode and then is
>>   executed normally.
>>
>> At first glance, I don't know what is "executed normally". The command?
>> The key? But a key isn't "executed" I believe, it is rather handled.
>
> Or more precisely: "The default key binding is used".

Sounds good.

>> And maybe it is also better to speak about "transient input mode", like
>> in other places in NEWS? This would give a better idea what it is about
>> to exit.
>
> A transient input method is something completely different.
>
>> (I'm not much familiar with "transient modes", and so will be most of
>> the readers of this NEWS file)
>
> From (info "(elisp) Controlling Active Maps"):
>
>  -- Function: set-transient-map keymap &optional keep-pred on-exit
>      This function adds KEYMAP as a “transient” keymap, which takes
>      precedence over other keymaps for one (or more) subsequent keys.
>
> So maybe "transient mode" should be replaced with "transient keymap"?

As said, I know almost nothing about "transient.*". The term "transient
mode" let me thing about a major or minor mode, which seems to be
wrong. Anything which prevents me (and others) to do so is fine :-)

Best regards, Michael.

Re: emacs-28 e55e2d4: ; * etc/NEWS: Minor copyedit about 'repeat-mode'.

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > Or more precisely: "The default key binding is used".

Could we make it active voice?

>   next-error matches.  Any other key exits transient mode and then
>   executes normally.

We should always use active voice, except when that's hard to do
for some reason.


-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)