58293 - plaintext compose doesn't wrap

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P3)

Description

[Akkana Peck]

Plaintext compose has stopped wrapping!  I thought it was problems related to my
signature (the signature is enclosed in a pre, and there are other known bugs
with trying to add text when the signature is the only thing in the document)
but I just turned off autosign and I'm still seeing no wrapping.  Of course, the
width of the mail window is variable, so the user can't tell where to wrap it
manually.

Comment 1

[Akkana Peck]

Correction: it looks like it is wrapping after all, but it's wrapping at 151
characters?!  Is this related to bug 58098?  (The window does in fact have
variable-width font.)

Comment 2

[Akkana Peck]

Yes indeedy, if I turn off the "always use my fonts" pref (which I had turned on
to test the fix for bug 58098), then plaintext compose wraps nicely.

Suggest we'd better relnote this.  Vera (or other relnote authors): the issue is
that if the user checks "always use my fonts", then the plaintext mail compose
window will display in the user's chosen *variable width* font, and will not
wrap at the user's selected wrap column.  The message that is sent will still be
wrapped correctly at send time.

Comment 3

[Ben Bucksch (:BenB)]

Akkana, do you run withteh patch applied? I noticed a comment obout wrapping
just above the lines that attinasi changed there.

Comment 4

[Akkana Peck]

I filed this bug based on the branch build, without the patch.  The patch should
fix it (on the trunk, but it's not on the branch, hence the request for release
notes).

Yes, the comment about wrapping refers to a bug where we were seeing exactly
this behavior, wrapping at some completely wacky place when using a variable
width font.  Apparently the width used for wrapping when the font is variable
width is very different from the actual average font width.  That's why we had
to add the "monospace" in the first place.

Comment 5

[Marc Attinasi]

Marking depends on fix in 58098. Accepting, thanks Akkana.

Comment 6

[Marc Attinasi]

Fixed on trunk, but not on branch. See comment <Akkana 2000-10-27 18:38> for
relnote information for branch build.

Leaving open for relnote...

Comment 7

[R.K.Aa.]

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

Comment 8

[R.K.Aa.]

Also reported on NT with a recent build. OS: All.

Comment 9

[Marc Attinasi]

Since the release is out, this can now be closed. Once again, this was fixed on
the trunk.

Comment 10

[Sheela Ravindran]

Verified on all