Bug 49501 - FORMATTING: RTF Formatting Issue
Summary: FORMATTING: RTF Formatting Issue
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version: 3.5.3 release
Hardware: Other All
: medium normal
Assignee: Miklos Vajna
QA Contact:
URL:
Whiteboard: BSA rtf_filter target:3.6.0 target:3.5.4
Keywords: regression
: 49413 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-05-04 10:43 UTC by David Baucum
Modified: 2013-12-05 15:19 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
This file shows the improper formatting. (23.67 KB, application/rtf)
2012-05-04 10:43 UTC, David Baucum
Details

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.