Bug 137949

Summary: IMPORT OOXML: Table row height is wrong.
Product: LibreOffice Reporter: Gülşah Köse <gulsah.kose>
Component: ImpressAssignee: Gülşah Köse <gulsah.kose>
Status: VERIFIED FIXED    
Severity: normal CC: aron.budea, buzea.bogdan, gerald, jluth, telesto, xiscofauli
Priority: medium Keywords: bibisected, bisected, regression
Version: 5.2 all versions   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=91762
https://bugs.documentfoundation.org/show_bug.cgi?id=119951
https://bugs.documentfoundation.org/show_bug.cgi?id=133935
https://bugs.documentfoundation.org/show_bug.cgi?id=139511
https://bugs.documentfoundation.org/show_bug.cgi?id=144092
Whiteboard: target:7.1.0 target:7.0.4 target:7.5.0 target:7.4.2
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 139899    
Attachments: sample
Comparison LibreOffice 7.1 master and MSO 2010

Description Gülşah Köse 2020-11-02 20:42:52 UTC
Description:
Table and row heights are different.

Steps to Reproduce:
1.Open sample.pptx

Actual Results:
Table height is bigger than actual. Row heights are not equal.

Expected Results:
Table height should be smaller, Row heights should be equal


Reproducible: Always


User Profile Reset: No



Additional Info:
-
Comment 1 Gülşah Köse 2020-11-02 20:43:32 UTC
Created attachment 166954 [details]
sample
Comment 2 Xisco Faulí 2020-11-03 09:16:47 UTC
Created attachment 166964 [details]
Comparison LibreOffice 7.1 master and  MSO 2010
Comment 3 Xisco Faulí 2020-11-03 09:38:37 UTC
this is a regression from my commit

author	Xisco Fauli <anistenis@gmail.com>	2016-01-21 20:56:31 +0100
committer	jan iversen <jani@documentfoundation.org>	2016-01-30 07:40:22 +0000
commit 13d4398820ded5914f635757865e258db2db2b57 (patch)
tree 356431bab06546113e07462070948388e3d6fc6e
parent 2416a2965f4d51e1b14b8539436f5f8fc16cc399 (diff)
tdf#91762: Check wether the row and the cell have text...

Bisected with: bibisect-linux-64-5.2
Comment 4 Gülşah Köse 2020-11-03 10:27:14 UTC
Hi Xisco, Can you review the patch when you have time? https://gerrit.libreoffice.org/c/core/+/105228
Comment 5 Commit Notification 2020-11-04 21:48:52 UTC
Gülşah Köse committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/7dc234fa57ca409d0db131c93abea738014b5e1f

tdf#137949 Fix table row heigths.

It will be available in 7.1.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 6 Xisco Faulí 2020-11-05 10:33:10 UTC
Verified in

Version: 7.1.0.0.alpha1+
Build ID: 9c8ed8c8526b9b696d0bf592eb7d963950f3cef4
CPU threads: 4; OS: Linux 5.7; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

@Gülşah Köse, thanks for fixing this issue!!
Comment 7 Gülşah Köse 2020-11-05 11:03:57 UTC
Thanks for the review Xisco o/
Comment 8 Commit Notification 2020-11-05 16:44:45 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/54daa022df46d07171201b3fe236cf3f5726e271

tdf#138011: Revert "tdf#137949 Fix table row heigths."

It will be available in 7.1.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 9 Xisco Faulí 2020-11-05 16:46:01 UTC
Commit reverted, reopening
Comment 10 Commit Notification 2020-11-09 11:36:56 UTC
Gülşah Köse committed a patch related to this issue.
It has been pushed to "master":

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

tdf#137949 Fix table row heigths.

It will be available in 7.1.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 11 Xisco Faulí 2020-11-09 14:10:15 UTC
*** Bug 135601 has been marked as a duplicate of this bug. ***
Comment 12 Commit Notification 2020-11-09 14:36:37 UTC
Gülşah Köse committed a patch related to this issue.
It has been pushed to "libreoffice-7-0":

https://git.libreoffice.org/core/commit/5c37f0c0347d8cccda6fc6a2ca4dd28bb36823a9

tdf#137949 Fix table row heigths.

It will be available in 7.0.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.
Comment 13 BogdanB 2021-01-28 06:16:09 UTC
It's ok now.

Verified in
Version: 7.0.4.2
Build ID: dcf040e67528d9187c66b2379df5ea4407429775
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 14 Commit Notification 2022-08-31 11:36:20 UTC
Sarper Akdemir committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/1a0ceb76e2fe12cdfff7cabf06ef43cfba296a34

tdf#144092 tdf#139511: revert parts of tdf#91762 tdf#137949 tdf#139511

It will be available in 7.5.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 15 Commit Notification 2022-09-05 12:24:25 UTC
Sarper Akdemir committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

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

tdf#144092 tdf#139511: revert parts of tdf#91762 tdf#137949 tdf#139511

It will be available in 7.4.2.

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.