Bug 136768 - FILEOPEN DOCX: page-gap in table until click in title(or lots of other things) for this particular file.
Summary: FILEOPEN DOCX: page-gap in table until click in title(or lots of other things...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.0.0.3 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected
Depends on:
Blocks: DOCX-Floatingtable
  Show dependency treegraph
 
Reported: 2020-09-15 11:42 UTC by Timur
Modified: 2023-09-15 14:53 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Timur 2020-09-15 11:42:24 UTC
Open 3 pages DOCX attachment 71442 [details]. It's 2007 DOCX but same if resaved in MSO.
LO opens 4 pages because 2nd page table is split to 3rd page. 
Upon click in title "Spørsmål for alle typer eiendommer", it comes back to 3 pages.
Comment 1 Timur 2020-09-15 11:53:34 UTC
Bibisect in 6.0 for table split - click in title couldn't fix there. 
 e5593290e3759d5cbe51a6a349d459651e3e6e81 is the first bad commit
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Thu Jul 13 02:04:23 2017 +0200

    source 25445d24cfa87522ee4c47e4aa7e6e816cdc9a36
    previous source 87633bd4e3a1b37ccfe1ddf18f1f178b07092a0d

commit 25445d24cfa87522ee4c47e4aa7e6e816cdc9a36	[log]
author	Miklos Vajna <vmiklos@collabora.co.uk>	Tue Jul 11 12:44:56 2017 +0200
committer	Miklos Vajna <vmiklos@collabora.co.uk>	Tue Jul 11 15:23:58 2017 +0200
tree b4b88a2fb5c52f6a98fc527f1b65c41e0c7cb582
parent 87633bd4e3a1b37ccfe1ddf18f1f178b07092a0d [diff]

tdf#109063 DOCX import: consider wrap space for multi-page floattables

freedesktop.org doesn't work to check if this is really the only commit, but let's assume it is, I provided also the previous commit. 

CC: Miklos.
Comment 2 Timur 2020-09-15 12:08:26 UTC
Another bibisect in 6.3 again for new table split, except this time click in title somehow joins the table. 
 4ea1db013d4b402c56758acc00182cf2702d31c0 is the first changed commit
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Thu Sep 26 02:58:11 2019 +0200

    source b0c5bc47d0d170df1384dd48cee9291ce6044083
    previous source 92267cdb1a45e1f40199136849feb692f7e06d0e

author	Justin Luth <justin_luth@sil.org>	2019-09-24 19:39:29 +0300
committer	Xisco Faulí <xiscofauli@libreoffice.org>	2019-09-25 11:56:54 +0200
commit b0c5bc47d0d170df1384dd48cee9291ce6044083 (patch)
tree 280e4bbbeb3306c9c36a62e9c756a151c5e77766
parent 92267cdb1a45e1f40199136849feb692f7e06d0e (diff)
Revert "tdf#117988 writerfilter: IgnoreTabsAndBlanksForLineCalculation"
This reverts LO 6.2 commit 49ddaad2f3ba4e17e1e41e94824fb94468d2b680.
tdf#127617 proves it simply was not the correct solution.

CC: Justin. 

So, we have two great devs ("usual suspects") here in this mess.
Comment 3 Justin L 2020-09-15 13:18:49 UTC
This will be some kind of layout bug.

Both of these commits will just have helped this particular docx trigger a certain condition that doesn't flow well. Neither one of these is truly a regression.

I don't notice anything particularly strange about this table/surrounding parargraphs. There are no "keep with next paragraphs" involved. The only strange thing is the first and last row are combined columns (gridSpan) - which I would expect to be irrelevant.

A few judiciously placed page break would have helped this document immensely.

And whatever is happening with those as-character shapes probably isn't helping either.
Comment 4 Telesto 2020-09-15 13:29:17 UTC
Not sure if bug 128437 (or duplicates are somehow related). Some behavior, different bibisect results. Maybe because of DOCX <-> ODT?
Comment 5 QA Administrators 2023-01-12 03:20:58 UTC Comment hidden (obsolete)
Comment 6 BogdanB 2023-04-22 08:52:52 UTC
Solved.

Verified with
Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 35787e07b7e83f7dcaa0c67830fcb4eded49c71f
CPU threads: 4; OS: Linux 6.2; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded
Comment 7 Timur 2023-09-15 14:10:23 UTC
Fixed with  source c31267e23603c77e3a3335fa431bb9b74448d5b2
author	Mike Kaganski <mike.kaganski@collabora.com>	Sat Jan 21 2023 +0300
tdf#153128: do not increase line height, when ignoring whitespace

I do not set duplicate as it did not look the same, this one was table.