221596 - HTML: unwanted wrapping of plain-text sig because sig's <pre> given cols=0

Status: NEW

(MailNews Core :: Composition, defect)

Description

[Biju]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030914 Firebird/0.6.1+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030914 Firebird/0.6.1+

If u have text signature and you try to create HTML mail.

ThunderBird append the signature insde following PRE tag 
<pre cols="0" class="moz-signature">--  </pre>

here cols="0" wrong, TB should calculate maximum line length 
and subsitute it.

or it should remove "cols" attribute completely.

see attachment for details

Temp. solution is in userContent.css add following

pre.moz-signature {
  width:100%  !important; 
  min-width:650px !important;
}


Reproducible: Always

Steps to Reproduce:
1. create a text signature 
2. set compose mail as HTML
3. open new mail window.



Actual Results:  
see attachemnt error.html

Expected Results:  
see attachemnt correct.html
or alternative.html

Mail send to Outlook user is view as in correct.html because Outlook ignore cols="0"

Temp. solution is in userContent.css add following

pre.moz-signature {
  width:100%  !important; 
  min-width:650px !important;
}

Comment 1

[Biju]

Attachment: error.html

Comment 2

[Biju]

Attachment: correct.html

Comment 3

[Biju]

Attachment: alternative.html

Comment 4

[Jeremy Lounds]

ANd, as an additional option, do not include the "---" line at all. That way, a
signature can start with "Thanks," "Sincerely,", etc. That would be helpful!

Comment 5

[Mike Cowperthwaite]

The 'cols=0' is being determined from the 'Wrap plain messages at...." 
preference (Tools | Options | Composition).

I agree it's not right for the zero case -- when that preference is zero, it 
means "wrap at window edge" not "wrap at the 0th column."

Arguably, setting 'cols' is not right even when the pref's value is normal (like 
72); but, there could be a problem if the sig contained a very long line.

Workaround: Select entire message, select  Insert|HTML  and then edit the 'cols' 
attribute directly.  Or, reset that preference to a typical value (72 or 80).

This occurs in Moz MailNews, as well.  =>Core

Comment 6

[Mike Cowperthwaite]

*** Bug 180451 has been marked as a duplicate of this bug. ***

Comment 8

[(not reading, please use [e-mail] instead)]

sorry for the spam.  making bugzilla reflect reality as I'm not working on these bugs.  filter on FOOBARCHEESE to remove these in bulk.