Closed (fixed)
Project:
Listhandler
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
14 Apr 2005 at 01:10 UTC
Updated:
27 Jul 2005 at 18:09 UTC
Emails containing a comment's contents, such as those listhandler sends, are particularly unfriendly because the default subject line, which is often not changed, is quite arbitrary. Using 'Re: parent's subject' is a bit better.
I did a bit of code cleanup around the lines I changed as well.
| Comment | File | Size | Author |
|---|---|---|---|
| comment.module_11.diff | 3.9 KB | drumm |
Comments
Comment #1
Steven commentedI thought the consensus was that Re subjects were evil? "Re" serves to indicate which message replies to what in emails. With threaded support in mail clients that's now becoming obsolete too, but in Drupal I see no need whatsoever to imitate this bad practice.
This line:
$edit['subject'] = truncate_utf8(decode_entities(strip_tags(check_output($edit['comment'], $edit['format']))), 29, TRUE);
is the result of a long discussion about comment subject autogeneration. In light of various input formats, mandatory / disallowed subjects, this seemed like the best idea.
Comment #2
factoryjoe commentedI think I want to +1 this, but I also think that the extra clutter in the web UI would be distracting. I think that comment titles should be optional -- and the email notifications that go out w/o a subject should grab the parent's subject like Drumm has done. So the real difference that I'm suggesting here is to allow for titleless comments, since people often write title-less comments anyway. Forcing a title only helps w/ email integration.
Chris
Comment #3
killes@www.drop.org commentedThis should be somehow handled by listhandler. Preferably by educating users to use proper MUAs which acn do threading without Re:s in subjects.
When in doubt use a bigger club.
Comment #4
killes@www.drop.org commentedOr get a consultant to implement your feature wishes. Done. Will be uplaoded later today.
Comment #5
(not verified) commentedComment #6
(not verified) commentedComment #7
drumm