unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#21303: 25.0.50; incorrect doc string for `menu-set-font'
@ 2015-08-20 15:39 Drew Adams
  2015-08-21 13:34 ` Eli Zaretskii
  0 siblings, 1 reply; 2+ messages in thread
From: Drew Adams @ 2015-08-20 15:39 UTC (permalink / raw)
  To: 21303

`C-h k' and choose `Set Default Font' from menu-bar menu `Options'.

The doc string says "Interactively select a font and make it the default
on all existing frames."

This is wrong.  The most important thing this command does is set the
font as the default for all FUTURE frames.  The doc string gives the
impression that it sets the default font only for the EXISTING frames.

Perhaps someone misread the doc string of `set-frame-font' when writing
the doc string of `menu-set-font', seeing only that arg FRAMES = t means
"all existing graphical frames".

The most important part of the behavior (and the doc string) of
`set-frame-font' is that a non-nil FRAME arg "alter[s] the user's
Customization settings as though the font-related attributes of the
`default' face had been "set in this session", so that the font is
applied to future frames."
^^^^^^^^^^^^^^^^^^^^^^^^

Users of `menu-set-font' should have this info, above all.


In GNU Emacs 25.0.50.1 (i686-pc-mingw32)
 of 2015-07-31 on LEG570
Bzr revision: 8d332aeccab2208e6c6bd434738565e6abf12043
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --host=i686-pc-mingw32 --enable-checking=yes,glyphs'





^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#21303: 25.0.50; incorrect doc string for `menu-set-font'
  2015-08-20 15:39 bug#21303: 25.0.50; incorrect doc string for `menu-set-font' Drew Adams
@ 2015-08-21 13:34 ` Eli Zaretskii
  0 siblings, 0 replies; 2+ messages in thread
From: Eli Zaretskii @ 2015-08-21 13:34 UTC (permalink / raw)
  To: Drew Adams; +Cc: 21303-done

> Date: Thu, 20 Aug 2015 08:39:49 -0700 (PDT)
> From: Drew Adams <drew.adams@oracle.com>
> 
> `C-h k' and choose `Set Default Font' from menu-bar menu `Options'.
> 
> The doc string says "Interactively select a font and make it the default
> on all existing frames."
> 
> This is wrong.  The most important thing this command does is set the
> font as the default for all FUTURE frames.  The doc string gives the
> impression that it sets the default font only for the EXISTING frames.

Thanks, fixed.





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-08-21 13:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-20 15:39 bug#21303: 25.0.50; incorrect doc string for `menu-set-font' Drew Adams
2015-08-21 13:34 ` Eli Zaretskii

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).