Bug 132539

Summary: FILEOPEN PPTX: "Perspective Below" shadow ("beveled oval" effect) shown above or not at all, depending on shadow size
Product: LibreOffice Reporter: Gerald Pfeifer <gerald>
Component: ImpressAssignee: Not Assigned <libreoffice-bugs>
Status: NEW ---    
Severity: normal CC: aron.budea, btomi96, gulsah.kose, xiscofauli
Priority: medium Keywords: bibisected, bisected
Version: 6.3.0.0.alpha0+   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=129916
https://bugs.documentfoundation.org/show_bug.cgi?id=130058
https://bugs.documentfoundation.org/show_bug.cgi?id=133473
https://bugs.documentfoundation.org/show_bug.cgi?id=140714
https://bugs.documentfoundation.org/show_bug.cgi?id=134210
Whiteboard:
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 120388    
Attachments: Sample PPTX document
How it should look like (Office 365)
This is how it looks in LO right now
This is how it looks like in 365's user interface
Sample compared MSO LO
Comparison MSO 2010 and LibreOffice 7.2 master
Sample PPTX with different Sizes
Sample PPTX with different Sizes compared in MSO 2016 and LO 7.5+

Description Gerald Pfeifer 2020-04-29 22:37:05 UTC
Created attachment 160098 [details]
Sample PPTX document

Receiving the original deck including of this slide I noticed that this
particular slide looked very oddly.  Investigating with Office 365 I
realized there is an effect labeled "beveled oval" that we are missing.
Comment 1 Gerald Pfeifer 2020-04-29 22:39:45 UTC Comment hidden (obsolete)
Comment 2 Gerald Pfeifer 2020-04-29 22:40:54 UTC Comment hidden (obsolete)
Comment 3 Gerald Pfeifer 2020-04-29 22:43:57 UTC
Created attachment 160101 [details]
This is how it looks like in 365's user interface
Comment 4 Xisco Faulí 2020-06-22 13:51:06 UTC
Reproduced in

Version: 7.1.0.0.alpha0+
Build ID: 70479e1f4cb3c120f46239a648e65f035af2922d
CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 5 Xisco Faulí 2020-06-22 13:55:58 UTC
The issue is visible since https://cgit.freedesktop.org/libreoffice/core/commit/?id=f4ba484183a1e7b9824f10580d633466c266828f when the images started to be displayed as circle. before that, they were displayed as squares.

@Tamas Bunth, i thought you might be interested in this issue...
Comment 6 Timur 2020-11-04 11:55:39 UTC
Created attachment 166999 [details]
Sample compared MSO LO

There was change in LO 7.0 (bad to bad). Seen in comparison here. 
In MSO I see this as Shadow  - Perspective: Below.  

There are more bugs on shadows, if this is not a duplicate, similar should be put in See Also.
Comment 7 Timur 2020-11-04 12:15:50 UTC

There were 2 changes in 7.0. 

1st change: 

commit 658e68588fd4ade17837026b28d9c2efd0898a27
Date:   Thu Apr 23 22:38:27 2020 +0200
    source 6454b6336b8de9a4c5899adeab552af6f794cdc4
    previous source 0e75f025419f878dc772391cea8d55765bceb9fc

author	Gülsah Köse <gulsah.kose@collabora.com>	2020-04-14 15:49:28 +0300
committer	Gülsah Köse <gulsah.kose@collabora.com>	2020-04-23 10:07:13 +0200
commit 6454b6336b8de9a4c5899adeab552af6f794cdc4 (patch)
tree efe9ea2e9b0131c4e9143cb1a085586de6f6e91a
parent 0e75f025419f878dc772391cea8d55765bceb9fc (diff)
tdf#130058 Import shadow size.

2nd change (same as in 7.1+: 
 de9c98af030cd9adcaac32fc4229af708e1386b9 is the first bad commit
commit de9c98af030cd9adcaac32fc4229af708e1386b9
Date:   Sat May 9 06:09:07 2020 +0200

    source 4259509aa4f11cbcd63299bd17a4503ebdc22fc0
    previous source 88173bd3ca4becc59282e58c035b882ca827a753

More commits: https://cgit.freedesktop.org/libreoffice/core/log/?qt=range&q=88173bd3ca4becc59282e58c035b882ca827a753..4259509aa4f11cbcd63299bd17a4503ebdc22fc0	
This one mentions shadow: 
author	Miklos Vajna <vmiklos@collabora.com>	2020-05-08 16:43:22 +0200
committer	Miklos Vajna <vmiklos@collabora.com>	2020-05-08 21:00:12 +0200
commit e2b0e614e1185c960b3015414919f69a1ed35aa0 (patch)
tree c4416c9047ba30bbd89c0fa09f3d2e177d7eaa2b
parent c35aef260feda34eae5e4b8a39f1baaa716b717d (diff)
Related: tdf#129916 svx: improve shadow size of custom shapes
Comment 8 Timur 2020-11-04 12:16:45 UTC
Gerald, these are all your bugs. Please follow and test with more complete sample, including all effects. It's pointless to go around like this.
Comment 9 Xisco Faulí 2021-04-26 19:02:30 UTC
in master, it looks much better after

https://cgit.freedesktop.org/libreoffice/core/commit/?id=ccdee8eebaa56619248e35001017226eecfe4e83

author	Gülşah Köse <gulsah.kose@collabora.com>	2021-02-01 17:03:33 +0300
committer	Gülşah Köse <gulsah.kose@collabora.com>	2021-02-10 07:04:38 +0100
commit ccdee8eebaa56619248e35001017226eecfe4e83 (patch)
tree 708db8ed2f0e37e9709b553b4a04de6c51a60aab
parent e4e0cf825a858e82c299c55632b03ca6f80647d1 (diff)
tdf#134210 Import crop position of bitmap filled shape.

The only problem left is the shadow, which is above, and not below
Comment 10 Xisco Faulí 2021-04-26 19:03:02 UTC
Created attachment 171434 [details]
Comparison MSO 2010 and LibreOffice 7.2 master
Comment 11 Xisco Faulí 2022-06-02 09:41:12 UTC
Still reproducible in

Version: 7.4.0.0.alpha1+ / LibreOffice Community
Build ID: 754eb1541a6ca709f78afbc7fb2b75f626562dcc
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 12 Timur 2022-07-19 11:38:00 UTC
Created attachment 181323 [details]
Sample PPTX with different Sizes
Comment 13 Timur 2022-07-19 11:41:12 UTC
Created attachment 181324 [details]
Sample PPTX with different Sizes compared in MSO 2016 and LO 7.5+
Comment 14 Timur 2022-07-19 11:42:55 UTC
Hi Gülsah. Since you already operated in a shadow .. please see :)
Comment 15 Gerald Pfeifer 2024-06-28 22:01:05 UTC
This actually looks fine now with

  Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
  Build ID: 97833217ebacd77d01b8e06ee1eb05cc06c17afc
  CPU threads: 12; OS: Linux 6.9; UI render: default; VCL: gtk3
  Locale: en-US (en_US.UTF-8); UI: en-US

except bug #161724 (a recent regression) hits and one of the two images
is completely gone.

Should we close this one and just leave bug #161724 open?