Bug 49501

Summary: FORMATTING: RTF Formatting Issue
Product: LibreOffice Reporter: David Baucum <maxolasersquad>
Component: WriterAssignee: Miklos Vajna <vmiklos>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium CC: f_rietveld, s-joyemusequna
Version: 3.5.3 releaseKeywords: regression
Hardware: Other   
OS: All   
Whiteboard: BSA rtf_filter target:3.6.0 target:3.5.4
i915 platform: i915 features:
Attachments: This file shows the improper formatting.

Description David Baucum 2012-05-04 10:43:03 UTC
Created attachment 61040 [details]
This file shows the improper formatting.

Problem description: 
Formatting problem.

Steps to reproduce:
1. Open the attached RTF.
2. Notice improperly formatted "Contractor Weekly Time Sheet"

Current behavior:
Formatting is incorrect.

Expected behavior:
Formatting should match LibreOffice 3.4 and below and other RTF viewers.

Platform (if different from the browser): 
              
Browser: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/535.19 (KHTML, like Gecko) Ubuntu/12.04 Chromium/18.0.1025.151 Chrome/18.0.1025.151 Safari/535.19
Comment 1 s-joyemusequna 2012-05-06 07:05:24 UTC
Works with LibO 3.4.5 (and Word 2007). Tested on Windows XP.
Doesn't work with LibO 3.5.3 (problem with table => 2 pages instead of one etc.) => REGRESSION
Comment 2 Miklos Vajna 2012-05-09 02:31:31 UTC
Confirmed, assign to me.
Comment 3 Not Assigned 2012-05-09 03:08:55 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=d6acd86fe1d2924a378e3053f83d47084a8bb108

fdo#49501 RTF_MARGL/R/T/B should also set the current margin
Comment 4 Miklos Vajna 2012-05-09 03:24:43 UTC
Fixed in master, closing - will request cherry-pick to  -3-5 a bit later.
Comment 5 Not Assigned 2012-05-11 08:38:00 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "libreoffice-3-5":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=71669deb856bcadba38f135837537ee8bd465f29&g=libreoffice-3-5

fdo#49501 RTF_MARGL/R/T/B should also set the current margin


It will be available in LibreOffice 3.5.4.
Comment 6 s-joyemusequna 2012-05-19 01:05:01 UTC
Tested with LOdev 3.6 (master - 18-May-2012 02h44 x86@6-fast; Build ID: 8b1d29b) and with LibO 3.5.4 RC1 under Windows Vista 64 and Windows XP.

It doesn't work. (It works with LibO 3.4.5 and Word 2007)
Comment 7 Miklos Vajna 2012-05-29 01:21:01 UTC
*** Bug 49413 has been marked as a duplicate of this bug. ***
Comment 8 Miklos Vajna 2012-05-30 01:43:28 UTC
Hi s-joyemusequna,

Ah, indeed - I only read your comment saying the margins are incorrect; the "Contractor Weekly Time Sheet" string is still not a single line. Could you please open a separate bug for that issue? Would be confusing to handle that in this bug, which talks about margin issues, fixed already in 3.5.4.

(So in short please reopen this only in case the doc starts to have 2 pages again.)

Thanks,

Miklos
Comment 9 s-joyemusequna 2012-05-30 04:21:01 UTC
Hi Miklos,

The document *has* 2 pages for me, the last one is empty, probably due to improperly formatted "Contractor Weekly Time Sheet". Tested with LibO 3.5.4 RC2 and LOdev 3.6 (master - 23-May-2012 03h12 x86@6-fast; Build ID: 7e65666) under Windows Vista 64 and Windows XP.

I did never say the margins were incorrect (you probably meant bug 49413 ?).

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.