Bug 104389 - Direct formatting instead of styles is used for text on table autoformat/styles
Summary: Direct formatting instead of styles is used for text on table autoformat/styles
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
5.2.3.3 release
Hardware: All All
: high normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
: 92235 115465 116378 133659 140183 145920 148199 (view as bug list)
Depends on: 49437
Blocks: Writer-Tables-Style Table-AutoFormat
  Show dependency treegraph
 
Reported: 2016-12-04 12:27 UTC by RGB
Modified: 2024-04-27 12:28 UTC (History)
13 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 RGB 2016-12-04 12:27:47 UTC
Tested with table autoformat on 5.2 and with table styles on 5.3 beta.

Steps to reproduce the problem:

1- Insert a table
2- With the cursor inside the table, apply a table autoformat (5.2) or table style (5.3 beta), for example, "lavender"

Results:

- Text on the table is set with Thorndale font even if the corresponding paragraph style is set as "liberation serif".

- Ctrl-M does NOT restore the paragraph style formatting, you need to apply it by hand.

Expected results: Table styles should not overwrite the paragraph styles used inside the table with direct formatting.
Comment 1 Regina Henschel 2016-12-04 22:34:06 UTC
Calc does not allow more than one paragraph in a cell and it uses cell styles. In Calc I see a similar problem. The "AutoFormat Style" applies a direct formatting of attributes, which were defined in a cell style in the source of that "AutoFormat Style". I guess, that the old binary autotbl.fmt is not able to contain styles. (But I don't know any documentation of that format). Writer uses the same old binary autotbl.fmt. In regard to the old binary format, I would consider this a "Wontfix", because the goal is to get rid of the binary format. 

The new table style in Writer uses the ODF element <table:table-template>. This has children, which are able to reference a cell-style. A cell-style can contain paragraph and text properties. But the Writer UI has no tools to define cell styles. So I think, that this is not really a bug, but a missing feature in the implementation.

LibreOffice would need an implementation of cell-styles in Writer (and in Impress/Draw too). And it would need a new UI for defining table styles including custom paragraph style and text style. Such UI is complex, because it is possible, that a cell can contain more than one paragraph and each paragraph can have a background in addition to the table-cells background.

Whether a table style should overwrite existing formating or not, when the table style is applied, is an additional problem.
Comment 2 Thomas Lendo 2018-02-06 13:42:47 UTC
Adding keyword needsUXEval and CC'ing Heiko and Jay as they worked on new table styles in Writer.

Please read Regina's comment 1, especially
(In reply to Regina Henschel from comment #1)
> The new table style in Writer uses the ODF element <table:table-template>.
> This has children, which are able to reference a cell-style. A cell-style
> can contain paragraph and text properties. But the Writer UI has no tools to
> define cell styles.
...
> LibreOffice would need an implementation of cell-styles in Writer (and in
> Impress/Draw too). And it would need a new UI for defining table styles
> including custom paragraph style and text style.

I also see a problem that table styles (old binary and new styles) ignore/don't use/don't set the table paragraphs "table content" and "table heading". Should table styles include paragraph styles or even overwrite paragraph/character attributes?
Comment 3 Heiko Tietze 2018-02-06 14:47:27 UTC
Table style is kind of a template rather than a real style. That means when you have defined the text as Thorndale it will be applied as such for the tables. 

The "legacy" Table > Autoformat Styles... dialog allows to select what property is used. This option is still missing together with the manipulation dialog for the new implementation. It was part of the design.
Comment 4 Heiko Tietze 2019-08-06 09:17:42 UTC
No need for input from UX. We have to get rid of the font properties in predefined table styles, which requires bug 49437 to be solved first.
Comment 5 Justin L 2020-07-04 09:43:59 UTC
*** Bug 92235 has been marked as a duplicate of this bug. ***
Comment 6 Dieter 2022-01-04 08:18:05 UTC Comment hidden (obsolete)
Comment 7 Heiko Tietze 2022-01-04 08:29:21 UTC
(In reply to Dieter from comment #6)
> Heiko, is there already a bug report for that enhancement?

Bug 105933 asks for "Means to modify existing table styles in a dialog", which should allow to not apply the PS.
Comment 8 Timur 2022-04-21 13:32:56 UTC
*** Bug 115465 has been marked as a duplicate of this bug. ***
Comment 9 Timur 2022-04-21 13:33:23 UTC
*** Bug 116378 has been marked as a duplicate of this bug. ***
Comment 10 Timur 2022-04-21 13:49:12 UTC
*** Bug 145920 has been marked as a duplicate of this bug. ***
Comment 11 Timur 2022-04-21 13:56:26 UTC
*** Bug 148199 has been marked as a duplicate of this bug. ***
Comment 12 Timur 2022-04-21 14:22:08 UTC
*** Bug 140183 has been marked as a duplicate of this bug. ***
Comment 13 Timur 2022-04-21 14:24:24 UTC
*** Bug 133659 has been marked as a duplicate of this bug. ***
Comment 14 Timur 2022-04-21 14:25:44 UTC
I marked bug 49437 as a Major and this one as High for duplicates and confusion.
Comment 15 QA Administrators 2024-04-27 03:16:50 UTC Comment hidden (obsolete)
Comment 16 RGB 2024-04-27 12:28:16 UTC
Problem still present on 

Version: 24.2.2.2 (X86_64) / LibreOffice Community
Build ID: d56cc158d8a96260b836f100ef4b4ef25d6f1a01
CPU threads: 8; OS: Linux 5.14; UI render: default; VCL: kf5 (cairo+xcb)
Locale: es-ES (es_ES.UTF-8); UI: es-ES
Calc: threaded

Steps to reproduce.

1) On a Writer document with the default template (default font: Liberation Serif), insert a table and fill it. Each cell will have the "Table content" paragraph style applied to it, with the default document font (again, Liberation Serif).

2) Apply any table style, for example "Elegant." You'll see the font change to Liberation Sans. 

3) Check the "Table contents" paragraph style: it's still set with Liberation Serif, but selecting the cells and doing a "clean direct formatting" don't fix the font.

4) Re-apply the "Table contents" paragraph style: fonts are "fixed."

5) Add either a row or a column to the table: fonts are changed again, ignoring the paragraph style settings.