* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
@ 2009-03-12 12:12 Leo
2009-03-12 13:45 ` Stefan Monnier
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Leo @ 2009-03-12 12:12 UTC (permalink / raw
To: emacs-pretest-bug
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@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.
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
2009-03-12 12:12 bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode Leo
@ 2009-03-12 13:45 ` Stefan Monnier
2009-03-12 17:36 ` Leo
[not found] ` <mailman.3019.1237105224.31690.bug-gnu-emacs@gnu.org>
2010-06-24 20:11 ` bug#2648: Deniz Dogan
2 siblings, 1 reply; 9+ messages in thread
From: Stefan Monnier @ 2009-03-12 13:45 UTC (permalink / raw
To: Leo; +Cc: 2648
> 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
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
2009-03-12 13:45 ` Stefan Monnier
@ 2009-03-12 17:36 ` Leo
2009-03-12 19:38 ` Stefan Monnier
0 siblings, 1 reply; 9+ messages in thread
From: Leo @ 2009-03-12 17:36 UTC (permalink / raw
To: bug-gnu-emacs
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
2009-03-12 17:36 ` Leo
@ 2009-03-12 19:38 ` Stefan Monnier
0 siblings, 0 replies; 9+ messages in thread
From: Stefan Monnier @ 2009-03-12 19:38 UTC (permalink / raw
To: Leo; +Cc: 2648
>> 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
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
[not found] ` <mailman.3019.1237105224.31690.bug-gnu-emacs@gnu.org>
@ 2009-03-15 12:35 ` Miles Bader
2009-03-15 13:09 ` Leo
2009-03-16 0:57 ` Stefan Monnier
0 siblings, 2 replies; 9+ messages in thread
From: Miles Bader @ 2009-03-15 12:35 UTC (permalink / raw
To: gnu-emacs-bug
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.
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
2009-03-15 12:35 ` Miles Bader
@ 2009-03-15 13:09 ` Leo
2009-03-16 0:57 ` Stefan Monnier
1 sibling, 0 replies; 9+ messages in thread
From: Leo @ 2009-03-15 13:09 UTC (permalink / raw
To: bug-gnu-emacs
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode
2009-03-15 12:35 ` Miles Bader
2009-03-15 13:09 ` Leo
@ 2009-03-16 0:57 ` Stefan Monnier
1 sibling, 0 replies; 9+ messages in thread
From: Stefan Monnier @ 2009-03-16 0:57 UTC (permalink / raw
To: Miles Bader; +Cc: gnu-emacs-bug, 2648
> 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
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648:
2009-03-12 12:12 bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode Leo
2009-03-12 13:45 ` Stefan Monnier
[not found] ` <mailman.3019.1237105224.31690.bug-gnu-emacs@gnu.org>
@ 2010-06-24 20:11 ` Deniz Dogan
2010-06-24 21:18 ` bug#2648: Glenn Morris
2 siblings, 1 reply; 9+ messages in thread
From: Deniz Dogan @ 2010-06-24 20:11 UTC (permalink / raw
To: 2648
It seems that the binding was removed in rev 98155 on October 15 2009.
^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#2648:
2010-06-24 20:11 ` bug#2648: Deniz Dogan
@ 2010-06-24 21:18 ` Glenn Morris
0 siblings, 0 replies; 9+ messages in thread
From: Glenn Morris @ 2010-06-24 21:18 UTC (permalink / raw
To: Deniz Dogan; +Cc: 2648-done
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@debbugs, as I did in this message.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2010-06-24 21:18 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-12 12:12 bug#2648: 23.0.60; remove C-c ` from rcirc track minor mode Leo
2009-03-12 13:45 ` Stefan Monnier
2009-03-12 17:36 ` Leo
2009-03-12 19:38 ` Stefan Monnier
[not found] ` <mailman.3019.1237105224.31690.bug-gnu-emacs@gnu.org>
2009-03-15 12:35 ` Miles Bader
2009-03-15 13:09 ` Leo
2009-03-16 0:57 ` Stefan Monnier
2010-06-24 20:11 ` bug#2648: Deniz Dogan
2010-06-24 21:18 ` bug#2648: Glenn Morris
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.