[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [address@hidden: Re: [Preview-latex-devel] CVS-1.42; scale function]
From: |
Stefan Monnier |
Subject: |
Re: [address@hidden: Re: [Preview-latex-devel] CVS-1.42; scale function] |
Date: |
08 Nov 2001 19:04:17 -0500 |
User-agent: |
Gnus/5.09 (Gnus v5.9.0) Emacs/21.1.50 |
>>>>> "David" == David Kastrup <David.Kastrup@t-online.de> writes:
> The mouse-2 feature already gets provided by a 'keymap property
> (actually, the 'local-map property since flyspell.el is lacking the
> knowledge that Emacs-21 already has the 'keymap property) on the
> overlay. The minor map key binding is there from a time where the
> overlay keymaps did not work, and so the entire buffer got overlayed
> with a mouse-2 binding that tries to detect whether the mouse is
> pressed in an overlay and if not, tries to do what the mousepress
> would have done instead. This fake key action ignores overlay keymaps
> and thus destroys my bindings.
Ah, I think you're right. It probably ends up not taking
the overlay bindings into account. I'll take a look.
Thank you for that precise description.
As for "why the minor mode map". It's actually a little bit more subtle
than just historical: the intent is to be able to type M-TAB even
a few chars after the mistyped word in which case the overlay's
`local-map' and `keymap' are of no help.
> flyspell is trying to simulate not being active at all, but fails. It
> is easier to let flyspell not attempt something which is absolutely
> and totally unnecessary than to try to improve its attempt at
> simulating Emacs.
I agree that the code is risky. See above for the reason why it's done
that way. As for whether the risk is worth the benefit, I don't know.
Richard> On the contrary, the `local-map' property is NEWER than
Richard> the `keymap' property.
Your memory is failing you, Richard. The `keymap' property is new
in Emacs-21.
Stefan
Re: [address@hidden: Re: [Preview-latex-devel] CVS-1.42; scale function], Stefan Monnier, 2001/11/07