Closed Bug 135163 Opened 23 years ago Closed 21 years ago

strange attribute on signatures added to HTML messages

Categories

(MailNews Core :: Composition, defect)

Product:
MailNews Core
Component:
Composition
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WORKSFORME

People

(Reporter: glazou, Assigned: sspitzer)

References

(Blocks 1 open bug)

Details

1. be sure to compose messages in HTML 2. be sure to attach a signature to your messages 3. send a test message to self 4. look at source of incoming test message 5. locate the PRE element representing the signature This PRE element has a strange attribute cols="$mailwrapcol" which seems to be totally unused in Mozilla. It could be here for compatibility reasons with another MUA but I can't really tell. If this is unused, it should be removed. Here is the source of a test message (body part only): <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta http-equiv="Content-Type" content="text/html;charset=ISO-8859-8-I"> <title></title> </head> <body> Test message ; signature's source has a strange cols attr on the pre.<br> <pre class="moz-signature" cols="$mailwrapcol">-- <br>Daniel Glazman Composer &amp; CSS Engine<br>Netscape Communications Corp. <a class="moz-txt-link-abbreviated" href="mailto:glazman@netscape.com">glazman@netscape.com</a><br>"Tous les &eacute;t&eacute;s, les Ib&egrave;res deviennent plus rudes." Ren&eacute; Goscinny<br></pre> <br> </body> </html>
Cc:ing Varada and Akkana since this has been introduced by Varada's fix for bug 71361, with a lot of comments from Akkana about it.
cols="$mailwrapcol" looks like a bug -- surely that isn't what was intended, and the intent was to actually get the wrap column setting (from the pref) and substitute it in? I'm somewhat surprised that it would be considered desirable to put wrapping in a signature anyway. If I make a plaintext signature I wrap it the way I wanted it to appear, rather than making it with long lines and expect the mailer to wrap it (a lot of people do ascii art in their signatures). But I guess that must be what the mail people decided in bug 71361.
-->varada
Assignee: ducarroz → varada
taking all of varada's bugs.
Assignee: varada → sspitzer
This bug appears to have been taken care of. Daniel Glazman, do you agree?
No response from reporter; =>WFM
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → WORKSFORME
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.