all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* 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.