* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
@ 2010-06-10 0:20 Drew Adams
2010-06-10 16:23 ` Juri Linkov
2016-08-21 20:46 ` Alex
0 siblings, 2 replies; 7+ messages in thread
From: Drew Adams @ 2010-06-10 0:20 UTC (permalink / raw
To: 6387
In isearch.el, we have this:
(define-key map "\M-\t" 'isearch-complete)
We should also have this, so `ESC TAB' works too:
(define-key map [escape tab] 'isearch-complete)
In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600)
of 2010-06-07 on 3249CTO
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (4.4) --no-opt --cflags -Ic:/xpm/include'
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
2010-06-10 0:20 bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete' Drew Adams
@ 2010-06-10 16:23 ` Juri Linkov
2010-06-10 16:34 ` Drew Adams
2010-06-10 19:36 ` Stefan Monnier
2016-08-21 20:46 ` Alex
1 sibling, 2 replies; 7+ messages in thread
From: Juri Linkov @ 2010-06-10 16:23 UTC (permalink / raw
To: Drew Adams; +Cc: 6387
> In isearch.el, we have this:
> (define-key map "\M-\t" 'isearch-complete)
>
> We should also have this, so `ESC TAB' works too:
> (define-key map [escape tab] 'isearch-complete)
I wonder why `ESC C-i' works in isearch, but `ESC TAB' doesn't work.
(There is no special key binding for `ESC C-i' in isearch.)
--
Juri Linkov
http://www.jurta.org/emacs/
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
2010-06-10 16:23 ` Juri Linkov
@ 2010-06-10 16:34 ` Drew Adams
2010-06-10 19:36 ` Stefan Monnier
1 sibling, 0 replies; 7+ messages in thread
From: Drew Adams @ 2010-06-10 16:34 UTC (permalink / raw
To: 'Juri Linkov'; +Cc: 6387
> > In isearch.el, we have this:
> > (define-key map "\M-\t" 'isearch-complete)
> >
> > We should also have this, so `ESC TAB' works too:
> > (define-key map [escape tab] 'isearch-complete)
>
> I wonder why `ESC C-i' works in isearch, but `ESC TAB' doesn't work.
> (There is no special key binding for `ESC C-i' in isearch.)
See this comment in the code:
;; To handle local bindings with meta char prefix keys, define
;; another full keymap. This must be done for any other prefix
;; keys as well, one full keymap per char of the prefix key. It
;; would be simpler to disable the global keymap, and/or have a
;; default local key binding for any key not otherwise bound.
(let ((meta-map (make-sparse-keymap)))
(define-key map (char-to-string meta-prefix-char) meta-map)
(define-key map [escape] meta-map))
(define-key map (vector meta-prefix-char t) 'isearch-other-meta-char)
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
2010-06-10 16:23 ` Juri Linkov
2010-06-10 16:34 ` Drew Adams
@ 2010-06-10 19:36 ` Stefan Monnier
1 sibling, 0 replies; 7+ messages in thread
From: Stefan Monnier @ 2010-06-10 19:36 UTC (permalink / raw
To: Juri Linkov; +Cc: 6387
> I wonder why `ESC C-i' works in isearch, but `ESC TAB' doesn't work.
> (There is no special key binding for `ESC C-i' in isearch.)
It probably has to do with the fact that isearch uses default-bindings
(i.e. bindings to the event gt which stands for "any event").
Such bindings prevent function-key-map (and similar hardcoded key
translations such as A -> a, S-left -> left, double-mouse-1 -> mouse-1,
...) from doing its job (since function-key-map is only used if the
untranslated sequence has no bindings which never happens thanks to the
default binding which always applies).
I've tried to change read-key-sequence such that a default binding is
only applied *after* function-key-map (or at least that a default
binding does not prevent function-key-map translations), but so far this
has failed.
Stefan
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
2010-06-10 0:20 bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete' Drew Adams
2010-06-10 16:23 ` Juri Linkov
@ 2016-08-21 20:46 ` Alex
2016-08-21 21:52 ` Drew Adams
1 sibling, 1 reply; 7+ messages in thread
From: Alex @ 2016-08-21 20:46 UTC (permalink / raw
To: Drew Adams; +Cc: 6387
"Drew Adams" <drew.adams@oracle.com> writes:
> In isearch.el, we have this:
> (define-key map "\M-\t" 'isearch-complete)
>
> We should also have this, so `ESC TAB' works too:
> (define-key map [escape tab] 'isearch-complete)
>
> In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600)
> of 2010-06-07 on 3249CTO
> Windowing system distributor `Microsoft Corp.', version 5.1.2600
> configured using `configure --with-gcc (4.4) --no-opt --cflags -Ic:/xpm/include'
>
ESC TAB works for me since at least 24.4. Does it work for you now?
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
2016-08-21 20:46 ` Alex
@ 2016-08-21 21:52 ` Drew Adams
2016-08-21 22:06 ` Nicolas Petton
0 siblings, 1 reply; 7+ messages in thread
From: Drew Adams @ 2016-08-21 21:52 UTC (permalink / raw
To: Alex; +Cc: 6387
> > In isearch.el, we have this:
> > (define-key map "\M-\t" 'isearch-complete)
> >
> > We should also have this, so `ESC TAB' works too:
> ESC TAB works for me since at least 24.4. Does it work for you now?
Yes. The bug was fixed in 24.4. Earlier versions had an explicit binding of <escape> TAB to `isearch-complete', but ESC TAB did not work.
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete'
2016-08-21 21:52 ` Drew Adams
@ 2016-08-21 22:06 ` Nicolas Petton
0 siblings, 0 replies; 7+ messages in thread
From: Nicolas Petton @ 2016-08-21 22:06 UTC (permalink / raw
To: Drew Adams, Alex; +Cc: 6387
[-- Attachment #1: Type: text/plain, Size: 154 bytes --]
Drew Adams <drew.adams@oracle.com> writes:
close 6387
thanks
> Yes. The bug was fixed in 24.4.
Great, I'm closing this bug report then.
Cheers,
Nico
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 512 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-08-21 22:06 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-06-10 0:20 bug#6387: 24.0.50; ESC TAB should be bound to `isearch-complete' Drew Adams
2010-06-10 16:23 ` Juri Linkov
2010-06-10 16:34 ` Drew Adams
2010-06-10 19:36 ` Stefan Monnier
2016-08-21 20:46 ` Alex
2016-08-21 21:52 ` Drew Adams
2016-08-21 22:06 ` Nicolas Petton
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.