Bug 136234 - FILEOPEN PPTX: Last entry in "diamond" SmartArt renders vertically, because width is tight, instead to go outside
Summary: FILEOPEN PPTX: Last entry in "diamond" SmartArt renders vertically, because w...
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
5.0.0.5 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: filter:pptx
Depends on:
Blocks: OOXML-SmartArt
  Show dependency tree / graph
 
Reported: 2020-08-28 16:07 UTC by Gerald Pfeifer
Modified: 2022-07-22 11:16 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:

Attachments
Sample slide (PPTX) (140.85 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2020-08-28 16:07 UTC, Gerald Pfeifer 		Details
Screenshot with LibreOffice 7.x (30.16 KB, image/png)
2020-08-28 16:08 UTC, Gerald Pfeifer 		Details
Screenshot with Office 365 (35.43 KB, image/png)
2020-08-28 16:09 UTC, Gerald Pfeifer 		Details
PPTX compared in MSO and LO (170.77 KB, image/png)
2020-09-14 10:27 UTC, Timur 		Details
Sample saved in MSO as PPT (254.50 KB, application/vnd.ms-powerpoint)
2020-09-14 10:40 UTC, Timur 		Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gerald Pfeifer 2020-08-28 16:07:47 UTC 
Created attachment 164815 [details]
Sample slide (PPTX)

This slide does not look great in PowerPoint/Office 365, but is readable.

In LibreOffice the text of the last entry renders vertically instead of
horizontally (which looks quite inconsistent).  Plus it renders much of
that white on white, which means its illegible.


Version: 7.1.0.0.alpha0+
Build ID: cac7ad2d011eb6602b6c9702dddb76d129f093c8
CPU threads: 4; OS: Linux 5.8; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2020-08-27_04:59:51
Calc: threaded

Version: 7.0.2.0.0+
Build ID: 5f305e6792c1c166b2a44a1e5085f42f53db50ea
CPU threads: 4; OS: Linux 5.8; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:libreoffice-7-0, Time: 2020-08-26_00:15:32
Calc: threaded
Comment 1 Gerald Pfeifer 2020-08-28 16:08:14 UTC 
Created attachment 164816 [details]
Screenshot with LibreOffice 7.x
Comment 2 Gerald Pfeifer 2020-08-28 16:09:15 UTC 
Created attachment 164817 [details]
Screenshot with Office 365
Comment 3 Xisco Faulí 2020-09-07 09:23:58 UTC 
Reproduced in

Version: 7.1.0.0.alpha0+
Build ID: 6b2eff7d69c6e14d89dd33eaa58c01d82c541266
CPU threads: 4; OS: Linux 5.7; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 4 Xisco Faulí 2020-09-07 09:24:59 UTC 
Also reproduced in

Version: 5.2.0.0.alpha0+
Build ID: 3ca42d8d51174010d5e8a32b96e9b4c0b3730a53
Threads 4; Ver: 5.7; Render: default;
Comment 5 Aron Budea 2020-09-11 23:06:33 UTC 
Actually, it renders horizontally in 4.4.0.3, though in two lines instead of one. Might be worth bibisecting, but I'm not marking regression, it was never good.
Still renders horizontally in latest of bibisect-50max, but already vertically in 5.0.0.5 / Ubuntu.
Comment 6 Timur 2020-09-14 10:27:26 UTC 
Created attachment 165481 [details]
PPTX compared in MSO and LO
Comment 7 Timur 2020-09-14 10:40:02 UTC 
Created attachment 165486 [details]
Sample saved in MSO as PPT

I remove bibisectRequest. Latest in 50max is old shape, oldest in 52 bibisect is new triangle shape. 
I don't find shape change relevant, it just was more space. 
Problem seems to be that Lo is trying to squeeze text into shape. 

Good is that PPT saved in MSO is read properly.
Comment 8 Timur 2020-09-14 11:13:55 UTC 
Key point is that bug happens only with grouped shape. Fine if ungrouped in MSO. 
Which makes it a probable duplicate of bug 59802.
Comment 9 Regina Henschel 2022-07-20 00:29:58 UTC 
The too small text area will be fixed with my work on bug 149551.
Comment 10 Regina Henschel 2022-07-20 11:04:40 UTC 
Fix is in https://cgit.freedesktop.org/libreoffice/core/commit/?id=7e23cbdbb6ec0247a29ed8a8f744c01e10963ea0

The fix will be included in the next daily build. Please test.
Comment 11 Gerald Pfeifer 2022-07-22 11:16:26 UTC 
(In reply to Regina Henschel from comment #10)
> The fix will be included in the next daily build. Please test.

Thank you, Regina! Happy to confirm that the text renders as expected
(cf. attachment 165481 [details] here and attachment 181376 [details] in bug #150103 ).

The only issue left is that the text renders white on white instead
of black on white, cf. bug #150103 which I just filed.


Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 3b7c18a579f3165c9d425d172d697f8978d6cd84
CPU threads: 8; OS: Linux 5.18; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-U