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.devel Subject: Re: Proposal for an improved `help-for-help' Date: Wed, 07 Apr 2021 19:10:55 +0300 Message-ID: <83mtua9isw.fsf@gnu.org> References: <838s7hxqkr.fsf@gnu.org> Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="15005"; mail-complaints-to="usenet@ciao.gmane.io" Cc: emacs-devel@gnu.org To: Stefan Kangas Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Wed Apr 07 18:12:58 2021 Return-path: Envelope-to: ged-emacs-devel@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 1lUAnZ-0003ll-Ob for ged-emacs-devel@m.gmane-mx.org; Wed, 07 Apr 2021 18:12:57 +0200 Original-Received: from localhost ([::1]:53328 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lUAnY-0002M5-EB for ged-emacs-devel@m.gmane-mx.org; Wed, 07 Apr 2021 12:12:56 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:48792) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lUAlh-0000lA-Fc for emacs-devel@gnu.org; Wed, 07 Apr 2021 12:11:01 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:49253) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lUAlh-0006cd-1c; Wed, 07 Apr 2021 12:11:01 -0400 Original-Received: from 84.94.185.95.cable.012.net.il ([84.94.185.95]:1390 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1lUAlg-0003NY-BH; Wed, 07 Apr 2021 12:11:00 -0400 In-Reply-To: (message from Stefan Kangas on Wed, 7 Apr 2021 10:18:23 -0500) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:267530 Archived-At: > From: Stefan Kangas > Date: Wed, 7 Apr 2021 10:18:23 -0500 > Cc: emacs-devel@gnu.org > > I intend to finish this up and push this to master in the next couple of > days. Please send any further comments soon, or we will have to > consider them after this lands on master. Thanks in advance. You mean, comments to the one and only patch you posted? I have some: > - (pop-to-buffer " *Metahelp*" nil t) > + (pop-to-buffer " *Help Overview*" nil t) The change in the buffer's name is gratuitous, so I'd prefer not doing that. Who knows what 3rd-party packages or user-local commands could depend on that? > +(defface help-for-help-binding-argument '((t :foreground "PaleGreen3")) > + "Face used for headers in the `help-for-help' buffer." > + :group 'help) Is this a good face definition for all the supported terminals and both the light and dark backgrounds? IMO, it looks too pale on the default light background in a GUI session, at least. And green shade is a strange color for this purpose anyway. > + '(("Getting Help" I'd use something like "Describe and Find Commands, Keys, Functions". > + ("Info Manuals" Just "Manuals" is better, IMO: newbies don't necessarily know what is "Info". > + ("Info-goto-emacs-key-command-node" "KEYS" > + "Show Emacs manual section for command bound to key") ^^^^^^^^^^^^^^^^^^^^^^^^ "for a key sequence" is better, IMO. > + ("info" nil "Show all included manuals") That's inaccurate. "Show top-level menu of all installed manuals", perhaps? > + ("info-display-manual" "MANUAL" "Show given manual") "given"? by whom? How about "Read a specific manual"? > + ("info-lookup-symbol" "SYMBOL" > + "Find symbol in Info manual for current programming language")) "Show description of symbol in pertinent manual" > + ("Misc Help" "Other Help commands" > + ("Help Files" "Misc" > + ("Language Help" "I18n Help" Thanks.