unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: storm@cua.dk (Kim F. Storm)
To: "Drew Adams" <drew.adams@oracle.com>
Cc: emacs-devel@gnu.org
Subject: Re: keypad-*-setup
Date: Tue, 18 Sep 2007 14:25:18 +0200	[thread overview]
Message-ID: <87ps0gtb2p.fsf@kfs-lx.testafd.dk> (raw)
In-Reply-To: <BNELLINCGFJLDJIKDGACAEMGCCAA.drew.adams@oracle.com> (Drew Adams's message of "Mon\, 17 Sep 2007 15\:30\:09 -0700")

"Drew Adams" <drew.adams@oracle.com> writes:

>> > 1. A nil ("No change (keep existing bindings)") value for the
>> > setup options seems to be a no-op.... Am I reading this wrong?
>>
>> I don't really remember why I included that choice.
>> Most likely it was a way to specify "use default bindings or whatever
>> the user has changed them to".
>
> Do you agree that it is a no-op, and can be removed (or am I missing
> something)?

What if user has once customized one of the keypad setups, and later
wants to revert to the "default setting" ... then there need to be a
setting which corresponds to that setting.  

So I don't think it is a no-op.  But this may be one of the things I
don't understand about Customize :-)

> OK. Then I suggest that we remove the `numeric' choice. Even if someone did
> implement what you suggest someday, that could be done by providing the
> "right" default value for the explicit decimal-point character (which the
> user could choose to override or not). Do you agree?

What about users who already has set keypad binding to numeric ?

> Perhaps whoever is finalizing this patch for CVS etc. can try that. I think
> it should be correct.

I think everybody expected you to finalize the patch :-)

But if you make another revision of the patch, I'll take a look at
finalizing it.

> The difficulty I saw was that the previous manual section, Function Keys,
> speaks of keys such as <kp-divide> as "keypad" keys, but keypad.el is not
> concerned with those. There are two different uses of the word "keypad" in
> the manual, one refers to all keys with prefix `kp-'; the other refers only
> to the 11 numeric keys 0-9 and `.'.

Those extra keys are not affected by the state of the numlock key.

That's also why I prefer to say "numeric keypad" rather than keypad.
Perhaps the package should have been called numlock.el, numpad.el or
some such.  But I guess that's too late now.

> There needs, I think, to be some explanation of the terminology to avoid
> confusion - the reader of the previous manual section will otherwise
> misunderstand what is said in this section. Feel free to propose something
> shorter to make this point.

I think you made a good attempt at explaining it - saying that it
applies only to 0..9 and decimal/del key.  To remove the possible
confusion one could just say: 

"The other keypad keys (such as kp-divide) are not affected by the
keypad customizations described here".

>> The Shift key and the NumLock key modify the behavior of keys on the
>> numeric keypad.  The Shift key acts as usual.  The NumLock key toggles
>> the keypad keys between two possible modes: numeric and non-numeric.
>
> Sounds good to me.

Ok.

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

  reply	other threads:[~2007-09-18 12:25 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-05  6:11 keypad-*-setup Drew Adams
2007-09-10  1:12 ` keypad-*-setup Richard Stallman
2007-09-10  1:24   ` keypad-*-setup Drew Adams
2007-09-10 23:54     ` keypad-*-setup Richard Stallman
2007-09-10 23:54     ` keypad-*-setup Richard Stallman
2007-09-11 12:13     ` keypad-*-setup Kim F. Storm
2007-09-11 19:51       ` keypad-*-setup Drew Adams
2007-09-11 21:54         ` keypad-*-setup Kim F. Storm
2007-09-12 17:22           ` keypad-*-setup Drew Adams
2007-09-13 16:29             ` keypad-*-setup Richard Stallman
2007-09-17 22:04             ` keypad-*-setup Kim F. Storm
2007-09-17 22:30               ` keypad-*-setup Drew Adams
2007-09-18 12:25                 ` Kim F. Storm [this message]
2007-09-18 15:11                   ` keypad-*-setup Drew Adams
2007-09-18  4:16               ` keypad-*-setup tomas

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=87ps0gtb2p.fsf@kfs-lx.testafd.dk \
    --to=storm@cua.dk \
    --cc=drew.adams@oracle.com \
    --cc=emacs-devel@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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).