Closed Bug 1367891 Opened 8 years ago Closed 8 years ago

Wordwrap no longer works in v. 52.1 in plain text compose

Categories

(Thunderbird :: Untriaged, defect)

Product:
Thunderbird
Component:
Untriaged
Version:
52 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WORKSFORME

People

(Reporter: coffent, Unassigned)

Details

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0 Build ID: 20170518000419 Steps to reproduce: Installed v. 52.1 and tried to write and read plain-text messages. Actual results: Lines don't wrap to the window. Expected results: Lines should wrap to the window. **Please see https://support.mozilla.org/en-US/questions/1161310#answer-971936 for a detailed discussion** Note I fixed the problem by reinstalling v. 45.8.
I've read through the support article. Yes, as you noticed in the release notes, some things have changed in plain text editing. However, the text still warps if you don't modify the default values. To shed some light onto this, install the add-on ThunderHTMLedit, then in a plain text window, go to the HTML tab (yes, plain text is just HTML behind the scenes!!) and paste what you see to a comment here. I see: <body style="font-family: -moz-fixed; white-space: pre-wrap; width: 72ch;"> So very clearly, the style is set to pre-wrap with a width of 72 characters. So that *does* wrap, I can see it here working. The only preference relevant here is mailnews.wraplength, so which value have you set there?
A plain text reply now looks like this: <body style="font-family: -moz-fixed; white-space: pre-wrap; width: 72ch;"> <div class="moz-cite-prefix">On 26/05/2017 10:09, someone wrote:<br></div> <span style="white-space: pre-wrap; display: block; width: 98vw;">&gt; quoted text<br></span> New text<br></body> That is, the width is 72 characters wide for new text, however, the quoted part will be wrapped to 98% of the screen width. We implemented that so that really long quoted lines get wrapped at the screen boundary and don't cause a scrollbar. See bug 387687, bug 1288911 and follow-ups bug 1328093 and bug 1330796.
OK, reading the support request, the reporter wants to wrap to the window, not the fixed 72 characters. To achieve this, you need to set mailnews.wraplength to 0. The result is: <body style="font-family: -moz-fixed; white-space: pre-wrap;"> Setting the preference to -1 results in: <body style="white-space: pre;"> so no wrapping. This works for me, so I'm closing the bug. Please reopen if you see something wrong with this.
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Summary: Wordwrap no longer works in v. 52.1 → Wordwrap no longer works in v. 52.1 in plain text compose
Maybe the reporter had mail.compose.wrap_to_window_width set which was removed in bug 233705.
You need to log in before you can comment on or make changes to this bug.