bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#2533: marked as done (23.0.90; [PATCH] rmail: improve rmail-add/kil


From: Emacs bug Tracking System
Subject: bug#2533: marked as done (23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring)
Date: Thu, 05 Mar 2009 05:35:05 +0000

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
--- Begin Message --- Subject: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Date: Mon, 2 Mar 2009 02:25:03 +0100 User-agent: Rmail GNU emacs 23.0 on Slackware 12.2.0
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



--- End Message ---
--- Begin Message --- Subject: Re: bug#2533: 23.0.90; [PATCH] rmail: improve rmail-add/kill-label docstring Date: Thu, 5 Mar 2009 00:26:32 -0500
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.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]