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.