From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: "Drew Adams" Newsgroups: gmane.emacs.devel Subject: RE: info.el patch to open node in new window Date: Sat, 29 Sep 2007 19:19:03 -0700 Message-ID: References: <873awxqin8.fsf@jurta.org> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Trace: sea.gmane.org 1191118816 28373 80.91.229.12 (30 Sep 2007 02:20:16 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Sun, 30 Sep 2007 02:20:16 +0000 (UTC) Cc: emacs-devel@gnu.org To: "Juri Linkov" Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sun Sep 30 04:20:12 2007 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1IboPi-0006Y7-QP for ged-emacs-devel@m.gmane.org; Sun, 30 Sep 2007 04:20:11 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IboPf-0002Uf-0y for ged-emacs-devel@m.gmane.org; Sat, 29 Sep 2007 22:20:07 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IboOz-0002Cj-Rb for emacs-devel@gnu.org; Sat, 29 Sep 2007 22:19:25 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IboOx-0002BE-IN for emacs-devel@gnu.org; Sat, 29 Sep 2007 22:19:24 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IboOw-0002Aj-Kx for emacs-devel@gnu.org; Sat, 29 Sep 2007 22:19:22 -0400 Original-Received: from agminet01.oracle.com ([141.146.126.228]) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1IboOw-0006SB-8I for emacs-devel@gnu.org; Sat, 29 Sep 2007 22:19:22 -0400 Original-Received: from rgmgw1.us.oracle.com (rgmgw1.us.oracle.com [138.1.186.110]) by agminet01.oracle.com (Switch-3.2.4/Switch-3.1.7) with ESMTP id l8U2JJCe019160; Sat, 29 Sep 2007 21:19:19 -0500 Original-Received: from acsmt350.oracle.com (acsmt350.oracle.com [141.146.40.150]) by rgmgw1.us.oracle.com (Switch-3.2.4/Switch-3.2.4) with ESMTP id l8U2JIWm013599; Sat, 29 Sep 2007 20:19:18 -0600 Original-Received: from dhcp-amer-whq-csvpn-gw3-141-144-80-240.vpn.oracle.com by acsmt351.oracle.com with ESMTP id 3255727441191118725; Sat, 29 Sep 2007 19:18:45 -0700 X-Priority: 3 (Normal) X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook IMO, Build 9.0.6604 (9.0.2911.0) Importance: Normal X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.3138 In-Reply-To: <873awxqin8.fsf@jurta.org> X-Brightmail-Tracker: AAAAAQAAAAI= X-Brightmail-Tracker: AAAAAQAAAAI= X-Whitelist: TRUE X-Whitelist: TRUE X-Detected-Kernel: Linux 2.4-2.6 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:80034 Archived-At: > > I'm not tied to a particular implementation, but I don't see what the > > problem is with this implementation. You've said 3 times now > > that FORK must go, but you haven't yet given any reason, AFAICT. > > > > I suggested these commands and bindings long ago. Please add > > them, however you want to implement them. > > Ok, the FORK argument of existing Info commands can't be removed for > compatibility reasons. I also think that adding the FORK argument > to all Info navigation commands like `u' (Info-up), `t' (Info-top-node), > `d' (Info-directory), etc makes no sense because they might make > a better use > of the prefix argument and it's just easy to type `M-n u', `M-n d' etc. > > However, I agree that `Info-follow-nearest-node' and > `Info-mouse-follow-nearest-node' > are exceptions since they are general commands to follow a "link" (i.e. > an Info reference or an Info menu/index item) and most browsers have a key > to open a link in another window, so user will except Emacs to do > the same. > I think Info-mouse-follow-nearest-node should be the last command where > we add the FORK argument. > > I noticed that in your patch the new command names have the `-new-window' > suffix. I think `-other-window' is a better suffix. Also perhaps > it should be bound to [S-mouse-2] instead of [S-down-mouse-2]. It sounds like the two of us, at least, generally agree about this. `-other-window': I guess you're thinking that is consistent with some other command names. This is a bit different in that in all cases it creates a new window. I don't have a strong opinion about the name, in any case. S-down-mouse-2: I use that because I generally have S-down-mouse-2 bound to something else that tracks mouse movement: `mouse-scan-lines-or-M-:'. Using the same binding here overrides cleanly and prevents interference. A tracking function needs to be bound to a down event (the up event removes the highlighting). I sent a patch to include `mouse-scan-lines-or-M-:' in Emacs on 2007-07-13, Subject "mouse.el patch to flash yank position and highlight line". It highlights the line under the pointer as long as the mouse button is depressed; the highlighting tracks the pointer.