all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 20466@debbugs.gnu.org
Subject: bug#20466: 25.0.50; REGRESSION in `isearch-mode-map': <backspace> is not translated to DEL
Date: Thu, 30 Apr 2015 07:11:00 -0700 (PDT)	[thread overview]
Message-ID: <c7e71c7d-5487-46a7-bcbb-a280b38401dd@default> (raw)
In-Reply-To: <<83383hu3dn.fsf@gnu.org>>

> The "why" part was answered by Artur, who pointed to the bug report
> which triggered that change.

That reports a problem.  It doesn't imply that the chosen fix is
the best fix.

> > It is an incompatible change, and I see nothing in NEWS about it.
> 
> It's not really incompatible, but I agree that it would be good to
> document the change, although I don't believe we document changes in
> keybindings in general.

> if you want to change the binding of DEL, you had better changed
> the binding of Backspace accordingly at the same time, because
> there's no guarantee Backspace will not be bound in any particular
> mode.  So why you are showing a use case that violates this simple
> rule?

That's certainly a useful rule for users to keep in mind, and it
applies to all key translations.  But it has not been necessary
in the past to take it into account for Isearch.  That's all.

Dunno whether such a rule/reminder is mentioned in the Emacs docs,
or even where it should be mentioned (key translations are not
something that most users read about).  Even if it were mentioned
in the doc about key translations, I doubt that would help most
users who might stumble on this.  Think of the many users who
have asked about shifted keys (a similar translation gotcha).

FWIW: I don't mind this change, personally (I don't bind `DEL' in
`isearch-mode-map').  I'm just pointing out that it might not be
expected, and that some users will likely scratch their heads
trying to figure out what's going on.

What the best thing to do about it is now (maybe nothing?), I
don't know.





       reply	other threads:[~2015-04-30 14:11 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <<a3b97f70-3780-421f-9a84-90a236c5e083@default>
     [not found] ` <<83383hu3dn.fsf@gnu.org>
2015-04-30 14:11   ` Drew Adams [this message]
2015-04-30  5:19 bug#20466: 25.0.50; REGRESSION in `isearch-mode-map': <backspace> is not translated to DEL Drew Adams
2015-04-30  9:30 ` Artur Malabarba
2015-04-30 13:55 ` Eli Zaretskii
2015-04-30 23:12   ` Stefan Monnier
2015-05-01  6:30     ` Eli Zaretskii
2015-05-01 18:03       ` Stefan Monnier
2015-05-01 18:43         ` Eli Zaretskii
2015-05-01 21:12           ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c7e71c7d-5487-46a7-bcbb-a280b38401dd@default \
    --to=drew.adams@oracle.com \
    --cc=20466@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.