Bug 148515 - FILEOPEN RTF Empty paragraph with direct formatting overrides style-defined font size
Summary: FILEOPEN RTF Empty paragraph with direct formatting overrides style-defined f...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.4.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Vasily Melenchuk (CIB)
URL:
Whiteboard: target:7.4.0 target:7.3.4
Keywords: bibisected, bisected, regression
Depends on:
Blocks: RTF-Character
  Show dependency treegraph
 
Reported: 2022-04-11 13:17 UTC by Gabor Kelemen (allotropia)
Modified: 2022-08-18 18:10 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Example RTF file from Word 2013 (57.84 KB, application/msword)
2022-04-11 13:17 UTC, Gabor Kelemen (allotropia)
Details
The example file in Word 2013 and Writer master (98.33 KB, image/png)
2022-04-11 13:18 UTC, Gabor Kelemen (allotropia)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Kelemen (allotropia) 2022-04-11 13:17:59 UTC
Created attachment 179460 [details]
Example RTF file from Word 2013

Attached example file has two tables, one in the header and its copy in the body.
The empty cells have Normal style with direct formatting of Arial font applied.
When opened in Writer these empty cells (and the paragraph end markers in non-empty cells) get a 12 pt font size, instead of the 10 from Normal style.
This makes the table rows a bit taller, which may change document layout in case of larger tables.

1, Open attached file in Writer
2, Notice the font size is 12 pt and there is more empty space between the letters in the non-empty cell and its top-bottom borders than in Word

Version: 7.4.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: cf4d5ed026c8799a70432a832a8a707c2e316216
CPU threads: 14; OS: Windows 10.0 Build 19044; UI render: default; VCL: win
Locale: en-US (hu_HU); UI: en-US
Calc: threaded Jumbo

Not yet in 7.3.0

Started for the table in body with:

https://git.libreoffice.org/core/+/494020efe6d29bc319c3abcd3245425e394674ec

author	Vasily Melenchuk <vasily.melenchuk@cib.de>	Sun Mar 20 08:48:17 2022 +0300
committer	Miklos Vajna <vmiklos@collabora.com>	Mon Mar 21 08:36:40 2022 +0100

tdf#131234: RTF import: extend default char props list

For the table in header it started with:
https://git.libreoffice.org/core/+/79b5ff92b17fde902c2f9db3e806f1c77a387dff

commit 79b5ff92b17fde902c2f9db3e806f1c77a387dff	[log]
author	Vasily Melenchuk <vasily.melenchuk@cib.de>	Mon Mar 21 13:54:28 2022 +0300
committer	Miklos Vajna <vmiklos@collabora.com>	Mon Mar 21 15:58:54 2022 +0100

tdf#118047: RTF import: better deduplication for header/footer

Adding CC to: Vasily Melenchuk
Comment 1 Gabor Kelemen (allotropia) 2022-04-11 13:18:36 UTC
Created attachment 179461 [details]
The example file in Word 2013 and Writer master
Comment 2 Xisco Faulí 2022-04-11 15:07:57 UTC
Reproduced in

Version: 7.4.0.0.alpha0+ / LibreOffice Community
Build ID: b812150696c574aea0a173d11f178e8d458b1a3e
CPU threads: 8; OS: Linux 5.10; UI render: default; VCL: gtk3
Locale: es-ES (es_ES.UTF-8); UI: en-US
Calc: threaded
Comment 3 Commit Notification 2022-04-26 22:56:39 UTC
Vasily Melenchuk committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/3a05acb8f0d94728ea6cbfd7a69dac6ffa7ffc68

tdf#148515: RTF filter: do not use char prop defaults for tables

It will be available in 7.4.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 4 Commit Notification 2022-04-27 08:46:08 UTC
Vasily Melenchuk committed a patch related to this issue.
It has been pushed to "libreoffice-7-3":

https://git.libreoffice.org/core/commit/b6deb063ac182b631dc9b61b4baca18744794ab1

tdf#148515: RTF filter: do not use char prop defaults for tables

It will be available in 7.3.4.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.