<<< Date Index >>>     <<< Thread Index >>>

Re: [PATCH] compose to sender



Hello Brian, and thank you for proposing this interesting feature,

 On Thursday, February 28, 2008 at 23:35:03 -0600, Brian Medley wrote:

> I have created a patch that starts a new message to the sender of the
> currently selected message.
>| <compose-sender> OP_COMPOSE_SENDER "compose new message to sender"

    The naming could be improved. OP_COMPOSE_SENDER works from index and
pager, while the whole OP_COMPOSE_* namespace is reserved for operations
specific to the compose menu. <compose-sender> is a very close variant
of <mail> "compose a new mail message", and the name should carry this
clearly. I would have proposed:

| <mail-sender> OP_MAIL_SENDER "compose a new message to the sender"

    However I really prefer Fabian's idea about keeping the <mail>
function only, but where the "To:" prompt would optionaly default to the
current sender. Prompt using the M_CLEAR mode where anything typed
instantly replaces the default. The only concern would be interactions
with $autoedit: I'd say this mandates a $mail_defaults_to_sender
boolean, no?

    Users can think of the <compose-sender> feature as a variant of
<mail> (to the selected sender), or as a variant of <reply> (without
quote, attribution, nor refs), at will. But from code POV, the <mail>
side would need less changes.


Bye!    Alain.
-- 
When you want to reply to a mailing list, please avoid doing so from a
digest. This often builds incorrect references and breaks threads.