Closed Bug 659312 Opened 9 years ago Closed 9 years ago

Use of PRE in html exporter produces undesired results

Categories

(Calendar :: Printing, defect, trivial)

Lightning 1.0b2
x86
Windows 7
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: tlaithwaite, Assigned: tlaithwaite)

Details

Attachments

(1 file)

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
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?
Version: unspecified → Lightning 1.0b2
Attachment #534765 - Flags: review?(philipp)
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.
Assignee: nobody → tlaithwaite
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
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.
Attachment #534765 - Flags: review?(philipp) → review+
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/e76c429aac9c>
-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Trunk
Backported to comm-miramar <http://hg.mozilla.org/releases/comm-miramar/rev/a280e3325271>
Target Milestone: Trunk → 1.0b4
You need to log in before you can comment on or make changes to this bug.