* bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
@ 2009-03-02 1:25 ` Xavier Maillard
2009-03-05 5:20 ` Glenn Morris
2009-03-05 5:35 ` bug#2533: marked as done (23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring) Emacs bug Tracking System
0 siblings, 2 replies; 4+ messages in thread
From: Xavier Maillard @ 2009-03-02 1:25 UTC (permalink / raw)
To: emacs-pretest-bug
When using keys 'a' or 'k' in a `rmail-summary' buffer, it is not
obvious user can type multiple labels as "comma-separated list of
labels". OTOH, it is clearly announced when user wants to
summarize by labels. The patch enclosed tries to clarify this.
Hope that'll help.
ChangeLog proposed:
2009-03-01 Xavier Maillard <xma@gnu.org>
* mail/rmailsum.el (rmail-summary-add-label)
(rmail-summary-kill-label): Fix docstrings. Indicate user can type
multiple comma-separted list of labels.
Patch:
diff --git a/lisp/mail/rmailsum.el b/lisp/mail/rmailsum.el
index 6940f3d..1ae0bd9 100644
--- a/lisp/mail/rmailsum.el
+++ b/lisp/mail/rmailsum.el
@@ -1542,7 +1542,8 @@ Interactively, empty argument means use same regexp used last time."
(defun rmail-summary-add-label (label)
"Add LABEL to labels associated with current Rmail message.
-Completion is performed over known labels when reading."
+Completion is performed over known labels when reading.
+LABEL may be a comma-separated list of labels."
(interactive (list (save-excursion
(set-buffer rmail-buffer)
(rmail-read-label "Add label"))))
@@ -1552,7 +1553,8 @@ Completion is performed over known labels when reading."
(defun rmail-summary-kill-label (label)
"Remove LABEL from labels associated with current Rmail message.
-Completion is performed over known labels when reading."
+Completion is performed over known labels when reading.
+LABEL may be a comma-separated list of labels."
(interactive (list (save-excursion
(set-buffer rmail-buffer)
(rmail-read-label "Kill label"))))
Recent input:
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> <up> <up> <up> <down> <down> <down>
<down> <down> <down> <up> C-x u C-x u C-SPC C-w C-SPC
C-a M-w C-x b <return> C-s C-s C-s C-s <return> <down>
<down> <down> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> C-y <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> C-SPC <right> <right> <right> <right>
<right> <right> C-w <right> <right> <right> <right>
<right> <right> <right> <right> C-x b x m a - s u <tab>
<return> a ' t e s t <return> k g n u <tab> - e m <tab>
<return> k ' t e <tab> <return> k e <backspace> t e
s <tab> t SPC <backspace> <backspace> <backspace> <backspace>
t e s t <return> k <return> <up> <down> SPC C-x C-b
<up> <up> <return> C-x 1 <right> C-SPC <right> <right>
<right> <right> <right> <right> <right> <right> <right>
<right> C-w <right> <right> <right> <right> <right>
<right> C-SPC <right> <right> <right> C-w <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> C-a C-SPC C-e <left> M-w <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <up> C-e <return> C-y <down> <up> C-SPC
<down> <left> C-w C-x C-s C-x v = <switch-frame> C-x
1 <down> M-k C-x 1 C-x 1 <right> <left> C-x 0 C-x b
C-g C-x 1 C-x b <return> C-x 1 C-l / b <switch-frame>
C-x 5 0 M-x e m <backspace> <backspace> r e p o r t
- e m <tab> <return>
Recent messages:
Mark activated
Updating buffer list...
Mark set
Updating buffer list...done
Commands: m, u, t, RET, g, k, S, D, Q; q to quit; h for help
Mark set [5 times]
Saving file /home/xma/usr/src/GITed/emacs/lisp/mail/rmailsum.el...
Wrote /home/xma/usr/src/GITed/emacs/lisp/mail/rmailsum.el
Finding changes in /home/xma/usr/src/GITed/emacs/lisp/mail/rmailsum.el...
call-interactively: Attempt to delete minibuffer or sole ordinary window
Quit
Xavier
--
http://www.gnu.org
http://www.april.org
http://www.lolica.org
^ permalink raw reply related [flat|nested] 4+ messages in thread
* bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
2009-03-02 1:25 ` bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Xavier Maillard
@ 2009-03-05 5:20 ` Glenn Morris
2009-03-05 5:35 ` bug#2533: marked as done (23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring) Emacs bug Tracking System
1 sibling, 0 replies; 4+ messages in thread
From: Glenn Morris @ 2009-03-05 5:20 UTC (permalink / raw)
To: Xavier Maillard; +Cc: 2533
Xavier Maillard wrote:
> When using keys 'a' or 'k' in a `rmail-summary' buffer, it is not
> obvious user can type multiple labels as "comma-separated list of
> labels".
I don't believe one can, or should. These commands are intended to
operate on a single label. There's no attempt to call
mail-parse-comma-list or anything like that. If you try to add mutiple
labels without spaces after the comma, it won't work right.
> OTOH, it is clearly announced when user wants to summarize by labels.
That's a different case. That command is designed to handle multiple
labels.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Processed: Re: bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
[not found] <18863.25224.639908.749606@fencepost.gnu.org>
2009-03-02 1:25 ` bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Xavier Maillard
@ 2009-03-05 5:35 ` Emacs bug Tracking System
1 sibling, 0 replies; 4+ messages in thread
From: Emacs bug Tracking System @ 2009-03-05 5:35 UTC (permalink / raw)
To: Glenn Morris; +Cc: Rmail Maintainers
Processing commands for control@emacsbugs.donarmstrong.com:
> tags 2533 notabug wontfix
bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
Tags were: patch
Tags added: notabug, wontfix
> stop
Stopping processing here.
Please contact me if you need assistance.
Don Armstrong
(administrator, Emacs bugs database)
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#2533: marked as done (23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring)
2009-03-02 1:25 ` bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Xavier Maillard
2009-03-05 5:20 ` Glenn Morris
@ 2009-03-05 5:35 ` Emacs bug Tracking System
1 sibling, 0 replies; 4+ messages in thread
From: Emacs bug Tracking System @ 2009-03-05 5:35 UTC (permalink / raw)
To: Glenn Morris
[-- Attachment #1: Type: text/plain, Size: 936 bytes --]
Your message dated Thu, 5 Mar 2009 00:26:32 -0500
with message-id <18863.25224.639908.749606@fencepost.gnu.org>
and subject line Re: bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
has caused the Emacs bug report #2533,
regarding 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)
--
2533: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=2533
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems
[-- Attachment #2: Type: message/rfc822, Size: 6388 bytes --]
From: Xavier Maillard <xma@gnu.org>
To: emacs-pretest-bug@gnu.org
Subject: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
Date: Mon, 2 Mar 2009 02:25:03 +0100
Message-ID: <200903020125.n221P3dY004861@zogzog.maillard.mobi>
When using keys 'a' or 'k' in a `rmail-summary' buffer, it is not
obvious user can type multiple labels as "comma-separated list of
labels". OTOH, it is clearly announced when user wants to
summarize by labels. The patch enclosed tries to clarify this.
Hope that'll help.
ChangeLog proposed:
2009-03-01 Xavier Maillard <xma@gnu.org>
* mail/rmailsum.el (rmail-summary-add-label)
(rmail-summary-kill-label): Fix docstrings. Indicate user can type
multiple comma-separted list of labels.
Patch:
diff --git a/lisp/mail/rmailsum.el b/lisp/mail/rmailsum.el
index 6940f3d..1ae0bd9 100644
--- a/lisp/mail/rmailsum.el
+++ b/lisp/mail/rmailsum.el
@@ -1542,7 +1542,8 @@ Interactively, empty argument means use same regexp used last time."
(defun rmail-summary-add-label (label)
"Add LABEL to labels associated with current Rmail message.
-Completion is performed over known labels when reading."
+Completion is performed over known labels when reading.
+LABEL may be a comma-separated list of labels."
(interactive (list (save-excursion
(set-buffer rmail-buffer)
(rmail-read-label "Add label"))))
@@ -1552,7 +1553,8 @@ Completion is performed over known labels when reading."
(defun rmail-summary-kill-label (label)
"Remove LABEL from labels associated with current Rmail message.
-Completion is performed over known labels when reading."
+Completion is performed over known labels when reading.
+LABEL may be a comma-separated list of labels."
(interactive (list (save-excursion
(set-buffer rmail-buffer)
(rmail-read-label "Kill label"))))
Recent input:
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> <up> <up> <up> <down> <down> <down>
<down> <down> <down> <up> C-x u C-x u C-SPC C-w C-SPC
C-a M-w C-x b <return> C-s C-s C-s C-s <return> <down>
<down> <down> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> C-y <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> <left> <left> <left> <left> <left> <left>
<left> <left> C-SPC <right> <right> <right> <right>
<right> <right> C-w <right> <right> <right> <right>
<right> <right> <right> <right> C-x b x m a - s u <tab>
<return> a ' t e s t <return> k g n u <tab> - e m <tab>
<return> k ' t e <tab> <return> k e <backspace> t e
s <tab> t SPC <backspace> <backspace> <backspace> <backspace>
t e s t <return> k <return> <up> <down> SPC C-x C-b
<up> <up> <return> C-x 1 <right> C-SPC <right> <right>
<right> <right> <right> <right> <right> <right> <right>
<right> C-w <right> <right> <right> <right> <right>
<right> C-SPC <right> <right> <right> C-w <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> C-a C-SPC C-e <left> M-w <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <up> C-e <return> C-y <down> <up> C-SPC
<down> <left> C-w C-x C-s C-x v = <switch-frame> C-x
1 <down> M-k C-x 1 C-x 1 <right> <left> C-x 0 C-x b
C-g C-x 1 C-x b <return> C-x 1 C-l / b <switch-frame>
C-x 5 0 M-x e m <backspace> <backspace> r e p o r t
- e m <tab> <return>
Recent messages:
Mark activated
Updating buffer list...
Mark set
Updating buffer list...done
Commands: m, u, t, RET, g, k, S, D, Q; q to quit; h for help
Mark set [5 times]
Saving file /home/xma/usr/src/GITed/emacs/lisp/mail/rmailsum.el...
Wrote /home/xma/usr/src/GITed/emacs/lisp/mail/rmailsum.el
Finding changes in /home/xma/usr/src/GITed/emacs/lisp/mail/rmailsum.el...
call-interactively: Attempt to delete minibuffer or sole ordinary window
Quit
Xavier
--
http://www.gnu.org
http://www.april.org
http://www.lolica.org
[-- Attachment #3: Type: message/rfc822, Size: 1756 bytes --]
From: Glenn Morris <rgm@gnu.org>
To: 2533-done@emacsbugs.donarmstrong.com
Subject: Re: bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring
Date: Thu, 5 Mar 2009 00:26:32 -0500
Message-ID: <18863.25224.639908.749606@fencepost.gnu.org>
tags 2533 notabug wontfix
stop
Glenn Morris wrote (on Thu, 5 Mar 2009 at 00:20 -0500):
> If you try to add mutiple labels without spaces after the comma, it
> won't work right.
And it certainly does not work in the kill case, unless you happen to
specify the labels in the same order as they appear.
It's just a coincidence that these commands may sometimes work for
multiple labels.
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-03-05 5:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <18863.25224.639908.749606@fencepost.gnu.org>
2009-03-02 1:25 ` bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Xavier Maillard
2009-03-05 5:20 ` Glenn Morris
2009-03-05 5:35 ` bug#2533: marked as done (23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring) Emacs bug Tracking System
2009-03-05 5:35 ` Processed: Re: bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Emacs bug Tracking System
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).