From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#46853: Confusing terminology "face height" instead of "font size" Date: Mon, 01 Mar 2021 21:46:18 +0200 Message-ID: <83y2f67j3p.fsf@gnu.org> References: <835z2a952x.fsf@gnu.org> Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="32938"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 46853@debbugs.gnu.org To: Stefan Kangas Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Mon Mar 01 20:47:24 2021 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lGoVm-0008LY-I8 for geb-bug-gnu-emacs@m.gmane-mx.org; Mon, 01 Mar 2021 20:47:22 +0100 Original-Received: from localhost ([::1]:58206 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lGoVl-0001Ri-Ck for geb-bug-gnu-emacs@m.gmane-mx.org; Mon, 01 Mar 2021 14:47:21 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:49948) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lGoVS-0001QJ-1Z for bug-gnu-emacs@gnu.org; Mon, 01 Mar 2021 14:47:02 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]:39326) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lGoVR-0007jW-QM for bug-gnu-emacs@gnu.org; Mon, 01 Mar 2021 14:47:01 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1lGoVR-0002qc-OX for bug-gnu-emacs@gnu.org; Mon, 01 Mar 2021 14:47:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 01 Mar 2021 19:47:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 46853 X-GNU-PR-Package: emacs Original-Received: via spool by 46853-submit@debbugs.gnu.org id=B46853.161462799210911 (code B ref 46853); Mon, 01 Mar 2021 19:47:01 +0000 Original-Received: (at 46853) by debbugs.gnu.org; 1 Mar 2021 19:46:32 +0000 Original-Received: from localhost ([127.0.0.1]:50872 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lGoUy-0002pv-7M for submit@debbugs.gnu.org; Mon, 01 Mar 2021 14:46:32 -0500 Original-Received: from eggs.gnu.org ([209.51.188.92]:60256) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lGoUw-0002pg-Aq for 46853@debbugs.gnu.org; Mon, 01 Mar 2021 14:46:30 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:52564) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lGoUq-0007VY-O2; Mon, 01 Mar 2021 14:46:24 -0500 Original-Received: from 84.94.185.95.cable.012.net.il ([84.94.185.95]:4724 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1lGoUo-0007nB-OE; Mon, 01 Mar 2021 14:46:23 -0500 In-Reply-To: (message from Stefan Kangas on Mon, 1 Mar 2021 13:31:44 -0600) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:201152 Archived-At: > From: Stefan Kangas > Date: Mon, 1 Mar 2021 13:31:44 -0600 > Cc: 46853@debbugs.gnu.org > > Agreed, so here's a tentative diff. It could probably use some > polishing though. Thanks. First, this consistently removes the reference to the default face, which loses information. For example: > - To increase the height of the default face in the current buffer, > -type @kbd{C-x C-+} or @kbd{C-x C-=}. To decrease it, type @kbd{C-x > -C--}. To restore the default (global) face height, type @kbd{C-x > -C-0}. These keys are all bound to the same command, > -@code{text-scale-adjust}, which looks at the last key typed to > -determine which action to take. > + To increase the font size in the current buffer, type @kbd{C-x C-+} This should be To increase the height of the default face's font, type ... Please also retain the reference to the default face in all the other places where it was mentioned before. > --- a/lisp/cus-face.el > +++ b/lisp/cus-face.el > @@ -73,9 +73,9 @@ custom-face-attributes > > (:height > (choice :tag "Height" > - :help-echo "Face's font height." > + :help-echo "Font size." Here' I suggest "Face's font size." There's no need to remove the reference to the face, since in Emacs a font is just one attribute of the face. > -Each positive or negative step scales the default face height by this amount." > +Each positive or negative step scales the default font size by this amount." There's no such thing as "the default font", please use "the size of the default face's font" instead. > - (user-error "Cannot %s the default face height more than it already is" > + (user-error "Cannot %s the font size any further" Please keep the reference to the default face here. > + +, = Increase default font size in current buffer by one step > + - Decrease default font size in current buffer by one step "Default font" again. > `:height' > > -VALUE specifies the relative or absolute height of the font. An > -absolute height is an integer, and specifies font height in units > -of 1/10 pt. A relative height is either a floating point number, > -which specifies a scaling factor for the underlying face height; > -or a function that takes a single argument (the underlying face > -height) and returns the new height. Note that for the `default' > -face, you must specify an absolute height (since there is nothing > -for it to be relative to). > +VALUE specifies the relative or absolute font size (height of the > +font). An absolute height is an integer, and specifies font height in > +units of 1/10 pt. A relative height is either a floating point > +number, which specifies a scaling factor for the underlying face > +height; or a function that takes a single argument (the underlying > +face height) and returns the new height. Note that for the `default' > +face, you must specify an absolute height (since there is nothing for > +it to be relative to). I don't understand the point of this change. My suggestion would be to leave the original text intact, and add to it a note saying that this affects the size of the font, not just its height. > --- a/lisp/play/gamegrid.el > +++ b/lisp/play/gamegrid.el > @@ -452,6 +452,7 @@ gamegrid-init-buffer > ;; Adjust the height of the default face to the height of the > ;; images. Unlike XEmacs, Emacs doesn't allow making the default > ;; face buffer-local; so we do this with an overlay. > + ;; FIXME: Is the above correct? See face-remap.el. No, this was written before we had face-remap.el. But that doesn't belong to this issue.