659312 - Use of PRE in html exporter produces undesired results

Status: RESOLVED FIXED

(Calendar :: Printing, defect)

Description

[Tom Laithwaite]

User-Agent:       Mozilla/5.0 (Windows NT 6.1) AppleWebKit/534.24 (KHTML, like Gecko) Chrome/11.0.696.68 Safari/534.24
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-GB; rv:1.9.2.17) Gecko/20110414 Lightning/1.0b2 Thunderbird/3.1.10


Pre tags are used in the html exporter to preserve formatting, but the pre tags are used for the entire event making the output small with unwrapped long lines. 



Reproducible: Always

Comment 1

[Tom Laithwaite]

Attachment: Patch to change from pre tags to style white-space:pre-wrap

I have created a simple patch to change the pre tags to style pre-wrap. Formatting is preserved, but long lines are still wrapped.

I tested this with some paragraphs of Lorem ipsum, with the first line of each tabbed in. This work for every paragraph except the first? Is leading whitespace removed?

Comment 2

[Philip Chee]

Hi Tom!

The preferred format for patch files is unified diff. In future please use -u8p. If your setup allows it, a git style patch and/or a Mercurial changeset patch is generally welcome.

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 534765 [details] [diff] [review]
Patch to change from pre tags to style white-space:pre-wrap

The patch works great, r=philipp.

I'll check this in later today.

Comment 4

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/e76c429aac9c>
-> FIXED

Comment 5

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Backported to comm-miramar <http://hg.mozilla.org/releases/comm-miramar/rev/a280e3325271>