| Summary: | FILEOPEN Multiple sections with different margins change 2-column layout to 4 in DOC | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Aron Budea <aron.budea> |
| Component: | Calc | Assignee: | Justin L <jluth> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ilmari.lauhakangas, jluth, xiscofauli |
| Priority: | medium | Keywords: | filter:doc |
| Version: | 4.4.0.3 release | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:6.3.0 target:6.2.1 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 107836, 112762 | ||
| Attachments: |
Sample DOC
PDF exported in Word Sample DOC PDF exported in Word Sample DOCX |
||
|
Description
Aron Budea
2018-09-27 03:50:14 UTC
Created attachment 145200 [details]
PDF exported in Word
Bibisected to the following commit using repo bibisect-44max. Might be difficult to fix. author Luboš Luňák <l.lunak@collabora.com> 2014-07-21 10:56:52 +0200 committer Luboš Luňák <l.lunak@collabora.com> 2014-07-21 11:02:04 +0200 ww8import create a pagedesc if continuous section changes margins (bnc#875383) This is similar to what writerfilter does. MSWord can have one page with several different margins, which are saved using continuous sections, which causes all kinds of trouble, because either we treat them as Writer sections, which means we lose some of the data, or we treat them as Writer page styles, which causes spurious page breaks if in the wrong place. Either option has its problems, but here it seems slightly better to go for keeping the data and hoping the page break will be in a place where a break will be anyway. Forgot the commit link. https://cgit.freedesktop.org/libreoffice/core/commit/?id=c5ed52b1cd6f22787c94bec035ceecf9e1da3271 Created attachment 145208 [details]
Sample DOC
I updated the sample so it's longer than one page by default, and there's a larger difference between the margins of the two sections.
Created attachment 145209 [details]
PDF exported in Word
Created attachment 145210 [details]
Sample DOCX
For reference, here's the DOCX file with the same content, now it's obvious that the margin change is lost in this case.
Perhaps the DOC importer has the better workaround, and the DOCX importer should be adjusted, but the 2-column layout becoming 4-column seems to be a bug. However, that's not because of this commit, as far as I can see.
Confirmed. Arch Linux 64-bit Version: 6.2.0.0.alpha0+ Build ID: 9a373521d7a328197a4bf9abeb0a981b7acba896 CPU threads: 8; OS: Linux 4.18; UI render: default; VCL: gtk3_kde5; Locale: fi-FI (fi_FI.UTF-8); Calc: threaded Built on 19 October 2018 proposed fix at https://gerrit.libreoffice.org/66151 Justin Luth committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/84fefd7c295fc05499ca222dff50c2fe4e0fb27e%5E%21 tdf#120145 ww8import: ignoreCols if section is inserted It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Confirmed the fix with attachment 145208 [details]. Áron: is there a report for the issue of the section starting on a new page? Arch Linux 64-bit Version: 6.3.0.0.alpha0+ Build ID: 6e3c4ae8bca898700cde9caaff43c3a8a61eebc2 CPU threads: 8; OS: Linux 4.20; UI render: default; VCL: gtk3; Locale: fi-FI (fi_FI.UTF-8); UI-Language: en-US Calc: threaded Built on 13 January 2019 (In reply to Buovjaga from comment #10) > Áron: is there a report for the issue of the section starting on a new page? Word's continuous section break is an alien concept in LO, it can change several document properties mid-page that LO can only do with a new page style (headers/footers, margin etc.). I guess bug 89297 could be considered as the general enhancement of providing a similar functinality. Justin, thanks for fixing the bug with the columns. Justin Luth committed a patch related to this issue. It has been pushed to "libreoffice-6-2": https://git.libreoffice.org/core/+/0e863f5529318e07f46568150e489cb0bef9b616%5E%21 tdf#120145 ww8import: ignoreCols if section is inserted It will be available in 6.2.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |