Bug 113198

Summary: FILEOPEN: PPTX: Incorrect alignment of text on images
Product: LibreOffice Reporter: Xisco Faulí <xiscofauli>
Component: ImpressAssignee: Not Assigned <libreoffice-bugs>
Status: VERIFIED FIXED    
Severity: normal CC: aron.budea, gerald, rb.henschel, xiscofauli, zolnaitamas2000
Priority: medium Keywords: bibisected, bisected, filter:pptx, regression
Version: 5.4.0.0.alpha0+   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=104722
https://bugs.documentfoundation.org/show_bug.cgi?id=129685
Whiteboard: target:6.5.0 target:6.4.0.1 target:6.3.5
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 108226    
Attachments: sample document
comparison MSO 2010 and LibreOffice 6.0

Description Xisco Faulí 2017-10-17 16:40:32 UTC
Created attachment 137059 [details]
sample document

Steps to reproduce:
1. Open attached document

Observed behaviour: text should have center alignment. it has left alignment instead.

Reproduced in

Version: 6.0.0.0.alpha0+
Build ID: 616f21db9e50a77b0c02dfb123f871a742f46216
CPU threads: 4; OS: Linux 4.10; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); Calc: group

[Bug found by office-interoperability-tools]
Comment 1 Xisco Faulí 2017-10-17 16:42:55 UTC
Created attachment 137060 [details]
comparison MSO 2010 and LibreOffice 6.0
Comment 2 Xisco Faulí 2017-10-17 16:44:27 UTC
Regression introduced by:

author	Tamás Zolnai <tamas.zolnai@collabora.com>	2016-12-17 15:11:06 (GMT)
committer	Tamás Zolnai <tamas.zolnai@collabora.com>	2016-12-17 16:39:55 (GMT)
commit 5694b4970e3af096130e172ac6968b10ed587a6f (patch)
tree 4e1285389cbd88c318beda765bf7becc0795a0cf
parent 2beb90a041bc99644330977f13bfaa10a05b9a39 (diff)
tdf#104722: Wrong text alignment when text box is smaller than the text inside
The default TextHorizontalAlignment is block. Setting
paragraph alignment (on UI) works when this text
alignment property is set to block (otherwise has no effect).
In general this works on this way, but the existing
algorithm did not work in those cases when the
text was bigger than the container shape.

Bisected with bibisect-linux-64-5.4

Adding Cc: to Tamás Zolnai
Comment 3 Xisco Faulí 2019-08-08 10:11:25 UTC
*** Bug 126747 has been marked as a duplicate of this bug. ***
Comment 4 Commit Notification 2019-11-28 00:48:05 UTC
Tamas Bunth committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/89f0af144c18efafe2573801641689a1432c0cae

tdf#113198 set default shape paragraph alignment..

It will be available in 6.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 5 Xisco Faulí 2019-11-28 10:09:28 UTC
Verified in

Version: 6.5.0.0.alpha0+
Build ID: 2bf687bebb30defe576ddb20c1c9680979ed3086
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US
Calc: threaded

@Tamas, thanks for fixing this issue! Should it be closed as RESOLVED now ?
Comment 6 Commit Notification 2019-11-28 11:08:29 UTC
Tamas Bunth committed a patch related to this issue.
It has been pushed to "libreoffice-6-4":

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

tdf#113198 set default shape paragraph alignment..

It will be available in 6.4.0.1.

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 7 Commit Notification 2019-11-28 13:08:12 UTC
Tamas Bunth committed a patch related to this issue.
It has been pushed to "libreoffice-6-3":

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

tdf#113198 set default shape paragraph alignment..

It will be available in 6.3.5.

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.