Bug 135731 - Writer: Table: Behaviour is wrong when Alow table to split rows between pages is checked (Checking and unchecking Keep with next paragraph seems to influence this behaviour) - See comment 3
Summary: Writer: Table: Behaviour is wrong when Alow table to split rows between pages...
Status: RESOLVED NOTABUG
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
3.3.0 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-08-13 19:50 UTC by BogdanB
Modified: 2021-12-21 20:10 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
video of the bug (1.69 MB, video/mp4)
2020-08-13 19:52 UTC, BogdanB
Details
the document (8.50 KB, application/vnd.oasis.opendocument.text)
2020-08-13 19:55 UTC, BogdanB
Details

Note You need to log in before you can comment on or make changes to this bug.
Description BogdanB 2020-08-13 19:50:49 UTC
Description:
See the video for an easy understand of the bug

When I check Keep with next paragraph just the next paragraph should remain with the table. What it seems to happend: The option to split the table between pages looks checked but as behaviour it is NOT checked.

Steps to reproduce.
1. Open my document (with a newly created table in 7.0, the same happedn with old versions, so the problem is old)
2. Be sure that Allow table to split between pages is checked
3. Keep with next paragraph to be checked
4. The table should move row by row to the next page

Expected result: The table should move row by row to the next page
Actual result: the table moves all, in spite of having Allow table to split between pages checked


Steps to Reproduce:
1. Open my document (with a newly created table in 7.0, the same happedn with old versions, so the problem is old)
2. Be sure that Allow table to split between pages is checked
3. Keep with next paragraph to be checked
4. The table should move row by row to the next page

Actual Results:
The table moves all, in spite of having Allow table to split between pages checked


Expected Results:
The table should move row by row to the next page, having Allow table to split between pages checked


Reproducible: Always


User Profile Reset: Yes



Additional Info:
Version: 6.4.5.2
Build ID: a726b36747cf2001e06b58ad5db1aa3a9a1872d6
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3; 
Locale: ro-RO (ro_RO.UTF-8); UI-Language: en-US
Calc: threaded
Comment 1 BogdanB 2020-08-13 19:52:56 UTC
Created attachment 164280 [details]
video of the bug

Video showing the bug

In fact that option keep with paragraph is very hard to use because of this bug. It is working just for some particular cases.

What I think it happened: in the code the option to Keep to next paragraph is unchecking something from the option to Allow the table to split. Something is interfering.
Comment 2 BogdanB 2020-08-13 19:55:10 UTC
Created attachment 164281 [details]
the document

It's not important the document in which you test this, but I attached something to help you test faster.
Comment 3 Rizal Muttaqin 2020-08-13 20:52:04 UTC
Verified

In my test

1. "Keep with next paragraph" UNCHECKED: table moved row by row to next page
2. "Keep with next paragraph" CHECKED: whole table moved to the next page
3. After simulating number 1 option a while until the table splitted across page, then I CHECKED "Keep with next paragraph" option and delete empty space above the table: The table moved back row by row. After that I add empty space again by pressing Enter above the table whole table moved to the next page.

Seems inconsistent

Version: 7.1.0.0.alpha0+
Build ID: <buildversion>
CPU threads: 4; OS: Linux 4.20; UI render: default; VCL: kf5
Locale: id-ID (id_ID.UTF-8); UI: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2020-08-09_14:07:53
Calc: threaded
Comment 4 BogdanB 2020-08-14 04:14:44 UTC
Also in
Version: 5.4.7.2
Build ID: c838ef25c16710f8838b1faec480ebba495259d0
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk2; 
Locale: ro-RO (ro_RO.UTF-8); Calc: group

Also in
Version 3.6.7.2 (Build ID: e183d5b)

So I supose is inherited from OOo.
Comment 5 BogdanB 2021-01-15 21:15:13 UTC
Also in
Version: 7.2.0.0.alpha0+
Build ID: 5adc93a9a9426ef79054751be2904896f787a8a2
CPU threads: 4; OS: Linux 5.8; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded
Comment 6 BogdanB 2021-04-21 17:10:13 UTC
Tested again in
Version: 7.2.0.0.alpha0+ / LibreOffice Community
Build ID: 93bb5fcba9e58eeeaaed9521c3bb4eecf9b91ac1
CPU threads: 4; OS: Linux 5.8; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded

The same problem, "Allow row to break across pages and columns" not working as it should.
Comment 7 BogdanB 2021-08-25 07:27:27 UTC
REPRO in
Version: 7.2.0.4 (x64) / LibreOffice Community
Build ID: 9a9c6381e3f7a62afc1329bd359cc48accb6435b
CPU threads: 4; OS: Windows 10.0 Build 19043; UI render: Skia/Raster; VCL: win
Locale: ro-RO (ro_RO); UI: en-US
Calc: threaded
Comment 8 Justin L 2021-09-24 13:16:20 UTC
I'd say this is working as expected. This is a TABLE property. So the entire table is going to try to keep with the next paragraph - as long as it all fits on one page - the same as all other uses of keep-with-next.

I suppose some could argue that this should be a "combination" setting that means one thing if the table cannot be split, and another thing if the table must not be split.

I'm sure this is NOTABUG, but I don't see any clarity in http://docs.oasis-open.org/office/v1.2/os/OpenDocument-v1.2-os-part1.html#__RefHeading__1419828_253892949 to guarantee that it means "the whole table must keep".
Comment 9 Justin L 2021-09-24 13:30:43 UTC
On second thought, I'm going to explicitly state that this is not a bug. There is no clear evidence that the current implementation is wrong, and changing it would break existing documents - which definitely would be wrong.

MS formats do not have such a concept of a table property to keep with next - so we can't look in that direction for clarity either.
Comment 10 BogdanB 2021-09-24 14:27:47 UTC
At least that option should be removed if not working as expected.
Comment 11 BogdanB 2021-09-24 14:31:05 UTC
I have find in the source you have mentioned:
------------------
The fo:keep-with-next attribute is usable with the following elements: <style:paragraph-properties> 17.6 and <style:table-properties> 17.15.

The values of the fo:keep-with-next attribute are auto or always.

-----------
So, the table properties could have auto or always to keep-with-next.
Comment 12 Justin L 2021-09-24 14:50:13 UTC
(In reply to BogdanB from comment #10)
> At least that option should be removed if not working as expected.

But it is working as expected. The table is keeping together with the next table. Since they don't all fit on the first page, the whole thing moves to the second page in order to keep together. (The keep with next property has NOTHING to do with table/row split).
Comment 13 Justin L 2021-09-24 15:50:28 UTC
P.S. This is working exactly the same way as paragraphs (which have widow/orphan settings). The entire paragraph will move ahead to the next page, not just the orphan amount.
Comment 14 BogdanB 2021-09-24 16:01:53 UTC
Justin, please see comment 3 and video from comment 1.
The problem is that this option is not consistent.

Imagine that you have to explain to someone new about this function in LibreOffice and you have situations lie in the video, or like was described in comment 3.
It is working but sometimes, not everytime... So, it is a bug somewhere.
Comment 15 Justin L 2021-09-24 16:12:07 UTC
Sure - there are lots of bug reports about tables and layout and keep-with-next in general, especially with reflow when modifying. That just makes this secondary issue a duplicate of something else.

But the subject line and the expected results from comment 0 are NOTABUG.
Comment 16 BogdanB 2021-09-25 09:38:32 UTC
I changed the bug title, I retested again, the bug is about Allow row to break which is checked but the rows don't break. And this seems to be influenced by the Keep with next paragraph (if it is checked or not). Now I hope the video from comment 1 is clear.
Comment 17 BogdanB 2021-12-21 20:10:24 UTC
Ok. Let's close this. The options are not usable together. Each one has its purpose and will be used separate.