Closed Bug 5811 Opened 25 years ago Closed 25 years ago

Text wrapping not working yet

Categories

(MailNews Core :: Backend, defect, P3)

All
Linux
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: scurtis, Assigned: rhp)

Details

We have text wrapping on our smoketest list of things that are supposed to be working now. It's not working in the 4_30 Linux or Windows builds. I tested by sending myself a long-lined message from the same build and viewed it once it was received. It ran off the right side of the display area.
So there's not just one horizontal scroll bar, there are two, one that you have to scroll to to see? This mechanism is horrible, IMHO, and I'm almost ready to mark this as invalid, because the line isn't actually getting truncated, on second look, but it's not wrapping. With this apparently unlimited scrolling to the right, I don't see how a line would ever wrap.
We could turn this bug into getting rid of that horizontal scroll bar and also getting rid of having two vertical scroll bars (when your msg text is longer than the height of the message pane). Let's wait for Phil's input.
I think this is an editor/mail integ issue in that the editor folks have to give us some API to set the PRE COLS in the DOM, and JF has to write some code which reads our outbound wrapping prefs and sets the number of COLS correctly. Reassigning to ducarroz for M6. cc kostello so he can see this dependency coming.
Assignee: ducarroz → phil
Are we talking about wrapping long line when reading a message or wrapping line during the composition of a new message? For me this bug is about reading message therefore it's not releated to message composition. I reassign the bug to phil.
Assignee: phil → rhp
Target Milestone: M6 → M7
Oh, ok, if we're talking about *reading* messages, then libmime is responsible for that (based on the mail.wrap_long_lines pref from the old code). Reassigning to rhp, but I don't think this is an M6 stopper, so moving to M7.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
This should be working. What I have done is change the default setting for this pref (mail.wrap_long_lines) to be true in the code, but you can set this in your prefs.js file and it should work fine. I will check in these changes later today. - rhp
QA Contact: 4080 → 4109
Status: RESOLVED → VERIFIED
Verified in the May 17 Mac, Win32, and Linux Seamonkey builds. Per Rhp, the default setting is to have view wrap long lines as falsed. I verified that once I add the pref user_pref("mail.wrap_long_lines", true); messages are wrapped when viewing in the message pane.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.