Closed Bug 23011 Opened 25 years ago Closed 25 years ago

Signature html is weird

Categories

(MailNews Core :: Composition, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: rhp)

Details

If you sent an mail with a signature you mail will look something like this:
<html><head></head>
<body>
<br>
-- <br><pre>
Henrik Gemal, gemal@dk.net
WebMail Innovator
Tele Danmark Internet</pre></body>
</html>

If you do the same thing with Netscape 4.7 you'll get:
<!doctype html public "-//w3c//dtd html 4.0 transitional//en">
<html>
<p>--
<br>Henrik Gemal, gemal@dk.net
<br>WebMail Innovator
<br>Tele Danmark Internet
<br>&nbsp;</html>

Why is the "<br><pre>" after the "--" ?
Hardware: PC → All
Assignee: ducarroz → rhp
reassign to Rich.
Status: NEW → ASSIGNED
Target Milestone: M13
What do you see as the problem? I think that you should have the <pre> for sigs
to preserve any alignment people may do with ASCII art, etc...

I'll make a change and put the "--" after the <pre>.

- rhp
I think I just wanted it this way:
<br>
<pre>
--
Henrik Gemal, gemal@dk.net
WebMail Innovator
Tele Danmark Internet
</pre>
</body>
</html>
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Cleaned this up...should be better now.

- rhp
henrik - can you verify this for me?
QA Contact: lchiang → gemal
This one is impossible to verify since whenever I enter the Account settings and 
add a signature and press OK and go back into Account settings the signature is 
gone....!
Must be another bug!
My signature is now being sent as this:

<pre>
--
Henrik Gemal, <a href="mailto:gemal@dk.net">gemal@dk.net</a>
Webmail Innovator
Tele Danmark Internet</pre>


Which seems correct...
Now my sig looks like this:, which seems correct:

<div class="signature"><pre>-- 
Henrik Gemal, <a class="txt-link txt-link-abbreviated" 
href="mailto:gemal@dk.net">gemal@dk.net</a>
Webmail Evangelist
Tele Danmark Internet

<a class="txt-link txt-link-freetext" 
href="http://gemal.dk/card/">http://gemal.dk/card/</a></pre></div>
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.