GNU bug report logs - #2648
23.0.60; remove C-c ` from rcirc track minor mode

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 2648 in the body.
You can then email your comments to 2648 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: emacs-pretest-bug <at> gnu.org
Subject: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Thu, 12 Mar 2009 12:12:01 +0000

Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the emacs-pretest-bug <at> gnu.org mailing list.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

rcirc track mode defines the following keys.

(define-key rcirc-track-minor-mode-map (kbd "C-c `") 'rcirc-next-active-buffer)
(define-key rcirc-track-minor-mode-map (kbd "C-c C-@") 'rcirc-next-active-buffer)
(define-key rcirc-track-minor-mode-map (kbd "C-c C-SPC") 'rcirc-next-active-buffer)

When this mode is enabled, those keys become GLOBAL and therefore can
easily step on bindings from other packages. One of them is AUCTeX.

I think it is entirely redundant to bind 'C-c `'. I'd propose this key
be removed when emacs is released. Otherwise it will confuse AUCTeX
users. Whenever there's an error, AUCTeX will ask users to type C-c ` to
get to the error, but instead this key will run
rcirc-next-active-buffer.

Message #10 received at 2648 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Leo <sdl.web <at> gmail.com>
Cc: 2648 <at> debbugs.gnu.org
Subject: Re: bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Thu, 12 Mar 2009 09:45:12 -0400

> rcirc track mode defines the following keys.

> (define-key rcirc-track-minor-mode-map (kbd "C-c `") 'rcirc-next-active-buffer)
> (define-key rcirc-track-minor-mode-map (kbd "C-c C-@") 'rcirc-next-active-buffer)
> (define-key rcirc-track-minor-mode-map (kbd "C-c C-SPC") 'rcirc-next-active-buffer)

> When this mode is enabled, those keys become GLOBAL and therefore can
> easily step on bindings from other packages. One of them is AUCTeX.

> I think it is entirely redundant to bind 'C-c `'. I'd propose this key
> be removed when emacs is released. Otherwise it will confuse AUCTeX
> users. Whenever there's an error, AUCTeX will ask users to type C-c ` to
> get to the error, but instead this key will run
> rcirc-next-active-buffer.

The Elisp manual says:

      The key sequences bound in a minor mode should consist of `C-c'
   followed by one of `.,/?`'"[]\|~!#$%^&*()-_+='.  (The other punctuation
   characters are reserved for major modes.)

So, I think only the C-c ` binding should stay and the others should
be removed.  As for the conflict with AUCTeX, I'm not sure what we
should do.  Maybe use some other key.  Arguably, buffer-local minor
modes should take precedence over global minor modes, in which case
AUCTeX's C-c ` binding would take precedence, but this is a "TODO"
feature, so it doesn't help us here.


        Stefan

Message #15 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Thu, 12 Mar 2009 17:36:52 +0000

On 2009-03-12 13:45 +0000, Stefan Monnier wrote:
>> rcirc track mode defines the following keys.
>
>> (define-key rcirc-track-minor-mode-map (kbd "C-c `") 'rcirc-next-active-buffer)
>> (define-key rcirc-track-minor-mode-map (kbd "C-c C-@") 'rcirc-next-active-buffer)
>> (define-key rcirc-track-minor-mode-map (kbd "C-c C-SPC") 'rcirc-next-active-buffer)
>
>> When this mode is enabled, those keys become GLOBAL and therefore can
>> easily step on bindings from other packages. One of them is AUCTeX.
>
>> I think it is entirely redundant to bind 'C-c `'. I'd propose this key
>> be removed when emacs is released. Otherwise it will confuse AUCTeX
>> users. Whenever there's an error, AUCTeX will ask users to type C-c ` to
>> get to the error, but instead this key will run
>> rcirc-next-active-buffer.
>
> The Elisp manual says:
>
>       The key sequences bound in a minor mode should consist of `C-c'
>    followed by one of `.,/?`'"[]\|~!#$%^&*()-_+='.  (The other punctuation
>    characters are reserved for major modes.)

It seems not many packages are consciously following this guideline.

> So, I think only the C-c ` binding should stay and the others should
> be removed.

Then ERC will also have to be changed.

> As for the conflict with AUCTeX, I'm not sure what we should do.
> Maybe use some other key.  Arguably, buffer-local minor modes should
> take precedence over global minor modes, in which case AUCTeX's C-c `
> binding would take precedence, but this is a "TODO" feature, so it
> doesn't help us here.

One way or another. Given that AUCTeX is one of the major editing
environments for LaTeX/TeX, it seems entirely unnecessary for a released
emacs to cause any pain to those users, particularly when this can be
fixed with minimal changes.

>         Stefan

-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

               www.git-scm.com
    git - the one true version control system

Message #20 received at 2648 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Leo <sdl.web <at> gmail.com>
Cc: 2648 <at> debbugs.gnu.org
Subject: Re: bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Thu, 12 Mar 2009 15:38:46 -0400

>> The key sequences bound in a minor mode should consist of `C-c'
>> followed by one of `.,/?`'"[]\|~!#$%^&*()-_+='.  (The other punctuation
>> characters are reserved for major modes.)
> It seems not many packages are consciously following this guideline.

Outside packages, indeed.  Bundled packages follow it a bit more
carefully, tho.

>> So, I think only the C-c ` binding should stay and the others should
>> be removed.
> Then ERC will also have to be changed.

Could very well be.

>> As for the conflict with AUCTeX, I'm not sure what we should do.
>> Maybe use some other key.  Arguably, buffer-local minor modes should
>> take precedence over global minor modes, in which case AUCTeX's C-c `
>> binding would take precedence, but this is a "TODO" feature, so it
>> doesn't help us here.

> One way or another. Given that AUCTeX is one of the major editing
> environments for LaTeX/TeX, it seems entirely unnecessary for a released
> emacs to cause any pain to those users, particularly when this can be
> fixed with minimal changes.

Of course.  The question is what the trivial change will be.


        Stefan

Message #27 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Miles Bader <miles <at> gnu.org>
To: gnu-emacs-bug <at> moderators.individual.net
Subject: Re: bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Sun, 15 Mar 2009 21:35:41 +0900

I'll note that I use C-c C-SPC more than probably any rcirc binding.

For typical usage, it's important that it be very easy to type, and that
it be bound globally.

Maybe there's another binding with such properties, I dunno, but please
don't change it unthinkingly.

-Miles

-- 
Non-combatant, n. A dead Quaker.

Message #32 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Sun, 15 Mar 2009 13:09:32 +0000

On 2009-03-15 12:35 +0000, Miles Bader wrote:
> I'll note that I use C-c C-SPC more than probably any rcirc binding.
>
> For typical usage, it's important that it be very easy to type, and that
> it be bound globally.
>
> Maybe there's another binding with such properties, I dunno, but please
> don't change it unthinkingly.

I have also used C-c C-SPC both in ERC in the past and now in rcirc. I
never noticed C-c ` until it failed AUCTeX.

> -Miles

-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

               www.git-scm.com
    git - the one true version control system

Message #37 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Miles Bader <miles <at> gnu.org>
Cc: 2648 <at> debbugs.gnu.org, gnu-emacs-bug <at> moderators.individual.net
Subject: Re: bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
Date: Sun, 15 Mar 2009 20:57:32 -0400

> I'll note that I use C-c C-SPC more than probably any rcirc binding.
> For typical usage, it's important that it be very easy to type, and that
> it be bound globally.

It'd be OK to provide such a binding under the control of a custom
variable (that would default to mil).  But by default, it's wrong to use
such a key binding.
Maybe a binding under C-x would be preferable.

We could even imagine to provide such a command more generically (on
the model of C-x `), i.e. a command that switches to the next "buffer
with activity", which could be a shell buffer, an IRC buffer, or any
other such buffer that mostly allows interaction with an
external process (or a network connection).


        Stefan

Message #45 received at 2648 <at> debbugs.gnu.org (full text, mbox):

From: Deniz Dogan <deniz.a.m.dogan <at> gmail.com>
To: 2648 <at> debbugs.gnu.org
Date: Thu, 24 Jun 2010 22:11:04 +0200

It seems that the binding was removed in rev 98155 on October 15 2009.

Message #50 received at 2648-done <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Deniz Dogan <deniz.a.m.dogan <at> gmail.com>
Cc: 2648-done <at> debbugs.gnu.org
Subject: Re: bug#2648:
Date: Thu, 24 Jun 2010 17:18:18 -0400

Deniz Dogan wrote:

> It seems that the binding was removed in rev 98155 on October 15 2009.

Thanks. Feel free to close a bug if you notice it should be closed.

You just send a mail to ###-done <at> debbugs, as I did in this message.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.