Summary: | FILEOPEN ODT->DOC/DOCX: Table height slightly larger compared to ODT (since 7.1) | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Telesto <telesto> |
Component: | Writer | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | NEW --- | ||
Severity: | minor | CC: | libreoffice, michael.stahl, raal |
Priority: | low | Keywords: | bibisected, bisected, filter:docx |
Version: | 7.1.0.0.alpha0+ | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 104444 | ||
Attachments: | Example file |
Description
Telesto
2021-08-10 12:23:34 UTC
Created attachment 174177 [details]
Example file
Confirm in Version: 7.3.0.0.alpha0+ / LibreOffice Community Build ID: c97bdca2d7abf30a99c45192b8f49c5bf76ca035 CPU threads: 4; OS: Linux 5.11; UI render: default; VCL: gtk3 Locale: cs-CZ (cs_CZ.UTF-8); UI: en-US Calc: threaded This seems to have begun at the below commit. Adding Cc: to Michael Stahl ; Could you possibly take a look at this one? Thanks 9495f24f8b825bbff3c6a73cacd7e8ee5f359150 is the first bad commit commit 9495f24f8b825bbff3c6a73cacd7e8ee5f359150 Author: Jenkins Build User <tdf@pollux.tdf> Date: Fri Sep 25 01:45:31 2020 +0200 source 38aa699f265c17548769aaa4f20e1ae35d18f202 https://git.libreoffice.org/core/+/38aa699f265c17548769aaa4f20e1ae35d18f202 tdf#134782 sw: split AddParaSpacingToTableCells flag in 2 Please set Importance. This is Minor at best, low to lowest. so if i revert the commit in comment #3, nothing changes. if i try the latest libreoffice-7-0, -6-4 still the same result. but with libreoffice-6-3 we get the same looks when loading the exported DOCX as when loading the ODT. however, it turns out that Word 2013 shows the DOCX exported by LO 6.3 differently than LO 6.3, and in the same way as current LO => LO 6.3 DOCX import is wrong so it looks like this never actually worked => removing regression the ODT file has: <config:config-item config:name="AddParaSpacingToTableCells" config:type="boolean">true</config:config-item> <config:config-item config:name="AddParaLineSpacingToTableCells" config:type="boolean">false</config:config-item> AFAIK Word doesn't support such a mode directly. there is some code in MSWordExportBase::OutputTextNode() that checks the spacing setting and clears a Lower-space ... but the paragraphs in the table don't have a RES_UL_SPACE so the test is skipped and adding ADD_PARA_LINE_SPACING_TO_TABLE_CELLS there doesn't help. sw/source/filter/ww8/wrtw8nds.cxx: if (!m_rDoc.getIDocumentSettingAccess().get(DocumentSettingId::ADD_PARA_SPACING_TO_TABLE_CELLS)) not sure what to do about this but it doesn't look obvious. (In reply to Timur from comment #4) > Please set Importance. This is Minor at best, low to lowest. Well page flow of documents on export is kind of important (at least to me). If you have a nice layout in ODT, but this this being screwed on export. The big question is more how likely is this to occur. I get the impression that this isn't regular, based on comment 5. But well that wasn't something to be known, before you demoted the bug. |