bug#59600: 28.1.90; Docstrings generated by `define-globalized-minor-mode' and `define-minor-mode' :global t are not same

bug#59600: 28.1.90; Docstrings generated by `define-globalized-minor-mode' and `define-minor-mode' :global t are not same

[Ihor Radchenko]

Hi,

I noticed inconsistency between minor modes defined via
`define-globalized-minor-mode' and via `define-minor-mode' with :global
t keyword.

The former have the following in their docstring:

"Toggle %s in all buffers.\n"
"%s is enabled in all buffers where `%s' would do it."

However, no indication is given that globalized minor mode defined via
`define-minor-mode' :global t is global.

For example, `auto-save-visited-mode' is a global minor mode, but no
clear indication of this fact is given. It did cause confusion for some
users.

Expected: :global t clearly indicates that the mode is global.

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

bug#59600: 28.1.90; Docstrings generated by `define-globalized-minor-mode' and `define-minor-mode' :global t are not same

[Eli Zaretskii]

> From: Ihor Radchenko <yantar92@posteo.net>
> Date: Sat, 26 Nov 2022 06:21:13 +0000
> 
> I noticed inconsistency between minor modes defined via
> `define-globalized-minor-mode' and via `define-minor-mode' with :global
> t keyword.
> 
> The former have the following in their docstring:
> 
> "Toggle %s in all buffers.\n"
> "%s is enabled in all buffers where `%s' would do it."
> 
> However, no indication is given that globalized minor mode defined via
> `define-minor-mode' :global t is global.
> 
> For example, `auto-save-visited-mode' is a global minor mode, but no
> clear indication of this fact is given. It did cause confusion for some
> users.
> 
> Expected: :global t clearly indicates that the mode is global.

Adding Stefan to the discussion.

bug#59600: 28.1.90; Docstrings generated by `define-globalized-minor-mode' and `define-minor-mode' :global t are not same

[Stefan Monnier via Bug reports for GNU Emacs, the Swiss army knife of text editors]

>> The former have the following in their docstring:
>> 
>> "Toggle %s in all buffers.\n"
>> "%s is enabled in all buffers where `%s' would do it."
>> 
>> However, no indication is given that globalized minor mode defined via
>> `define-minor-mode' :global t is global.

[ Nitpick: "define-minor-mode :global t" does not (usually) define
  a "globalized mode" but rather it defines a "global mode".  ]

>> For example, `auto-save-visited-mode' is a global minor mode, but no
>> clear indication of this fact is given. It did cause confusion for some
>> users.
>> 
>> Expected: :global t clearly indicates that the mode is global.
>
> Adding Stefan to the discussion.

Not much to say, but I agree it makes sense for the docstring to state
the scope of the mode (global, buffer-local, younameit).


        Stefan

bug#59600: 28.1.90; Docstrings generated by `define-globalized-minor-mode' and `define-minor-mode' :global t are not same

[Eli Zaretskii]

> From: Stefan Monnier <monnier@iro.umontreal.ca>
> Cc: Ihor Radchenko <yantar92@posteo.net>,  59600@debbugs.gnu.org
> Date: Sat, 26 Nov 2022 12:38:33 -0500
> 
> [ Nitpick: "define-minor-mode :global t" does not (usually) define
>   a "globalized mode" but rather it defines a "global mode".  ]
> 
> >> For example, `auto-save-visited-mode' is a global minor mode, but no
> >> clear indication of this fact is given. It did cause confusion for some
> >> users.
> >> 
> >> Expected: :global t clearly indicates that the mode is global.
> >
> > Adding Stefan to the discussion.
> 
> Not much to say, but I agree it makes sense for the docstring to state
> the scope of the mode (global, buffer-local, younameit).

Patches to the macro that defines the modes are welcome.