Bug 143347 - Inserting a new row resets table formats.
Summary: Inserting a new row resets table formats.
Status: RESOLVED DUPLICATE of bug 126008
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.0.0.0.beta1+
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisectRequest, regression
Depends on:
Blocks: Table-Styles
  Show dependency treegraph
 
Reported: 2021-07-14 08:08 UTC by taiwuco
Modified: 2021-08-26 07:54 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
The screen recording of the bug (2.53 MB, video/x-matroska)
2021-07-14 08:08 UTC, taiwuco
Details

Note You need to log in before you can comment on or make changes to this bug.
Description taiwuco 2021-07-14 08:08:17 UTC
Created attachment 173551 [details]
The screen recording of the bug

This bug has existed for a long time.
When inserting a row or a column into a formatted table, the formats will be reset.
Comment 1 Telesto 2021-07-14 08:22:25 UTC
Repro
Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 4e0a7df2dfa10bc52d5dbda34f43e0bc0df22ae7
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

But only with table styles active (so in this case with default table style)
Comment 2 Telesto 2021-07-14 08:26:26 UTC
Found in
Version: 7.0.0.0.beta1+ (x64)
Build ID: 2891e91a513520d68ea2b8c59c14335861a15253
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

Not in
Version: 6.4.0.0.beta1+ (x64)
Build ID: 20be5cd0bdc57d812bf34a2debfe48caa51de881
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: default; VCL: win; 
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL
Comment 3 Balázs Sántha 2021-07-21 07:54:03 UTC Comment hidden (no-value)
Comment 4 Aron Budea 2021-07-26 01:24:53 UTC
(In reply to Telesto from comment #2)
> Not in
> Version: 6.4.0.0.beta1+ (x64)
> Build ID: 20be5cd0bdc57d812bf34a2debfe48caa51de881
Not sure this is correct, this bug looks like bug 126008, already in 6.1. Closing this as a dupe for now, if you have evidence this is different, feel free to reopen.

*** This bug has been marked as a duplicate of bug 126008 ***
Comment 5 taiwuco 2021-07-26 12:49:29 UTC
Thanks for notifying. I did make a quick search before posting the bug.

I read the bug 126008. I am no C++ developer but I, as a software developer, know the binary table autoformat is a dead end and this bug cannot be solved by debugging binary autoformat issue. It's too troublesome.

Why not just get rid of table autoformat? Instead, add a new format group tool to the table tool sets. Mark a group of formatted rows or columns as a format group in advance. Inserting a single row or column belonging to a format group will instead inserting the group of rows of columns. Deletion or other table tools apply the same concept.

I now use a no-style table only just to bypass the table autoformat.
Comment 6 Telesto 2021-07-27 01:05:18 UTC
(In reply to Aron Budea from comment #4)
Sorry, not a duplicate.. But thanks for finding the "see also" which I did know existed (but to lazy to search for) 

Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 2a151d1d5bc055d5e0011460b6ec42ea9f34f880
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL
Comment 7 Telesto 2021-07-27 01:16:35 UTC
@Heiko
The comment below is slightly off for fixing the bug here (I think). However the bug reporter is [properly, no dev] right about the "binary table autoformat is a dead" part. There must be some bug reported related to that, right? Searching proper ticket. [Assuming you keeping tabs on the matter combined with a vague recollection about some comments about this in context of issues arisen in the 5.3 branch]


(In reply to taiwuco from comment #5)
> Thanks for notifying. I did make a quick search before posting the bug.
> 
> I read the bug 126008. I am no C++ developer but I, as a software developer,
> know the binary table autoformat is a dead end and this bug cannot be solved
> by debugging binary autoformat issue. It's too troublesome.
> 
> Why not just get rid of table autoformat? Instead, add a new format group
> tool to the table tool sets. Mark a group of formatted rows or columns as a
> format group in advance. Inserting a single row or column belonging to a
> format group will instead inserting the group of rows of columns. Deletion
> or other table tools apply the same concept.
> 
> I now use a no-style table only just to bypass the table autoformat.
Comment 8 Aron Budea 2021-07-27 11:00:40 UTC
(In reply to Telesto from comment #6)
> Sorry, not a duplicate.. But thanks for finding the "see also" which I did
> know existed (but to lazy to search for) 
Care to elaborate?
Comment 9 taiwuco 2021-07-27 15:20:08 UTC
(In reply to Telesto from comment #7)
> There must be some bug reported related to that,
> right? Searching proper ticket.

Following the Bug 126008, the long-lasting Bug 107555 was referred to. That bug was first reported on 2017-05-02 and about table autoformat problem.
Comment 10 Heiko Tietze 2021-08-23 09:54:07 UTC
(In reply to Telesto from comment #7)
> There must be some bug reported related to that...

It's bug 49437. The issue here is that a direct formatting on top of a table styles is lost under some circumstances when the table changes. This wouldn't change with a non-binary format. The issue is that TS are not real styles and need a better implementation, see bug 34391. And there are plenty of issues reported so I guess this one has a duplicate anyway.
Comment 11 Buovjaga 2021-08-26 07:54:43 UTC
Checked the repos and it is already in 7.0, 6.4, so let's indeed close

*** This bug has been marked as a duplicate of bug 126008 ***