Gnus development mailing list
 help / color / mirror / Atom feed
From: Ted Zlatanov <tzz@lifelogs.com>
To: ding@gnus.org
Subject: Re: [PATCH] Introduce gnus-completing-read
Date: Tue, 28 Sep 2010 15:07:14 -0500	[thread overview]
Message-ID: <87tyl91vkd.fsf@lifelogs.com> (raw)
In-Reply-To: <1285688153-19680-1-git-send-email-julien@danjou.info>

On Tue, 28 Sep 2010 17:35:53 +0200 Julien Danjou <julien@danjou.info> wrote: 

JD> As promised yesterday, here's my patch!

JD> It adds `gnus-completion-styles' which set `completion-styles' when calling
JD> `completing-read'.

JD> It adds `gnus-use-ido' which makes `gnus-completing-read' use
JD> `ido-completing-read' rather than `completing-read'

JD> It replaces (almost) all occurences of `completing-read' with
JD> `gnus-completing-read', so it works with ido too.

Thanks for the patch, I think it's very useful.  Are there any specific
completion styles and behavior (e.g. `gnus-use-ido') you'd recommend?  I
don't use ido so I don't know how it's useful.  Some documentation would
be helpful, even if minimal, for what your patch does in the manual.  I
think it should go in the Gnus news file, too, because it's a pretty
major change across Gnus.

The default gnus-completion-styles appends 'substring to whatever else
the user has if it's not present (the docstring should reflect that).
Should we add 'partial-completion as well?  Or 'initials?  There seem to
be many places where gnus-completing-read makes sense with a different
completion style from the normal one.

For example the "lower score" command now asks for a series of choices
one by one, which IMO could be better done with completing-read choices.
In that case, 'initials would work well to represent `L s s p' (lower
subject, substring, permanent) as the completion choice
"lower-subject-substring-permanent" that the user can input as `lssp'.

Ted




  parent reply	other threads:[~2010-09-28 20:07 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-28 15:35 Julien Danjou
2010-09-28 15:49 ` Eric Abrahamsen
2010-09-28 15:55 ` Florian Ragwitz
2010-09-28 16:04   ` Julien Danjou
2010-09-28 19:31     ` Florian Ragwitz
2010-09-28 20:47       ` Julien Danjou
2010-09-29 15:19         ` Lars Magne Ingebrigtsen
2010-09-29 21:47           ` Lars Magne Ingebrigtsen
2010-09-30  8:05             ` Julien Danjou
2010-09-30  9:42               ` Robert Pluim
2010-09-30  9:56                 ` Julien Danjou
2010-09-30  6:46           ` Reiner Steib
2010-09-30 10:23             ` Florian Ragwitz
2010-09-30 10:44               ` Lars Magne Ingebrigtsen
2010-09-30 11:33                 ` Florian Ragwitz
2010-09-30 11:36                   ` Lars Magne Ingebrigtsen
2010-09-30 11:42                     ` Florian Ragwitz
2010-09-30 12:20                       ` Julien Danjou
2010-09-30 12:54                         ` Florian Ragwitz
2010-09-30 13:29                           ` Julien Danjou
2010-09-30 16:33                           ` Lars Magne Ingebrigtsen
2010-09-30 16:48                             ` Ted Zlatanov
2010-09-30 16:55                               ` Lars Magne Ingebrigtsen
2010-09-30 17:13                                 ` Ted Zlatanov
2010-09-30 17:22                                   ` Florian Ragwitz
2010-09-30 17:39                                     ` Lars Magne Ingebrigtsen
2010-09-30 17:41                                       ` Florian Ragwitz
2010-09-30 17:49                                         ` Lars Magne Ingebrigtsen
2010-10-01  1:01                                           ` Florian Ragwitz
2010-10-01  9:43                                             ` Lars Magne Ingebrigtsen
2010-10-01 10:13                                               ` Lars Magne Ingebrigtsen
2010-09-30 17:47                                     ` Ted Zlatanov
2010-10-01  9:00                                       ` Julien Danjou
2010-09-30 17:06                               ` Florian Ragwitz
2010-10-01  6:17                           ` Copyright assignments, policies, conventions (was: [PATCH] Introduce gnus-completing-read) Reiner Steib
2010-10-01 13:51                             ` Copyright assignments, policies, conventions Štěpán Němec
2010-09-30 12:13                   ` [PATCH] Introduce gnus-completing-read Štěpán Němec
2010-09-28 15:55 ` Dan Christensen
2010-09-28 16:00   ` Julien Danjou
2010-09-28 16:02     ` Julien Danjou
2010-09-28 20:07 ` Ted Zlatanov [this message]
2010-09-28 20:36   ` Julien Danjou
2010-09-28 21:08     ` Ted Zlatanov
2010-10-01 15:57 ` Matt Lundin
2010-10-01 18:21   ` Lars Magne Ingebrigtsen
2010-10-01 23:45     ` Matt Lundin
2010-10-02  2:21       ` Lars Magne Ingebrigtsen
2010-10-02 17:48         ` Matt Lundin

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=87tyl91vkd.fsf@lifelogs.com \
    --to=tzz@lifelogs.com \
    --cc=ding@gnus.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.
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 NNTP newsgroup(s).