Bug 67297 - rendering errors of embedded OLE previews (and EMF+'s of them ?)
Summary: rendering errors of embedded OLE previews (and EMF+'s of them ?)
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
(earliest affected) rc
Hardware: Other All
: medium major
Assignee: Muthu
Keywords: regression
Depends on:
Blocks: mab4.1
  Show dependency treegraph
Reported: 2013-07-25 11:15 UTC by Michael Meeks
Modified: 2013-10-11 06:34 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:

sample document (53.67 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2013-07-25 11:15 UTC, Michael Meeks
Minimal reproducer of bnc#812793 (78.21 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2013-10-05 19:01 UTC, Andras Timar

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Meeks 2013-07-25 11:15:21 UTC
It seems that we are incorrectly rendering either blue or transparent backgrounds on these EMF+ previews - which seems unfortunate, and makes many slides with linked (but not present) content look poor :-)
Comment 1 Michael Meeks 2013-07-25 11:15:50 UTC
Created attachment 82991 [details]
sample document
Comment 2 Caolán McNamara 2013-08-23 13:52:03 UTC
This is a regression caused by http://cgit.freedesktop.org/libreoffice/core/commit/?id=4bc3ead0649bf2885050ec4cfb47a5ed13bfa41e which was added to fix 
https://bugzilla.novell.com/812793‎ (which I don't have permissions to view)
Comment 3 Michael Meeks 2013-08-23 15:37:32 UTC
Muthu ? :-) You sure that's right - I would imagine we need a white background on these ?
Comment 4 Muthu 2013-08-30 15:31:38 UTC
Yes, I was sure about that document - checked quite a lot of stuffs in that document. Let me recheck...not sure how many documents are broken either ways :(

Clearly we are missing something or some setting in the document.
Comment 5 Andras Timar 2013-10-05 19:01:06 UTC
Created attachment 87168 [details]
Minimal reproducer of bnc#812793

I attached the minimal reproducer of bnc#812793, I hope this helps. The big pie chart on the left is the 5th p:pic in p:spTree, so it is placed on the top of the small pie charts (p:pic 1-4). I think the problem is, that a:clrChange of the big pie chart is not processed properly. It should turn white into fully transparent white.

If I reorder the p:pic elements, I get correct result without Muthu's patch. If I give a:alpha val=100% instead of 0, the big pie chart overlaps the small ones in PowerPoint 2010, too (and with val=50% it is 50% transparent, etc.). Impress ignores this element/attribute.
Comment 6 Muthu 2013-10-10 09:19:22 UTC
Andras: Thank you!
[I also did some importing with the emf as well - yes, this is with respect to the clrChange not being handled and not the emf itself]

[for completeness: Its not about ordering]

Reverting the workarounds.