[Libreoffice-bugs] [Bug 130058] FILEOPEN PPTX: image shadow that should be invisible shown as extraneous line below

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=130058

Aron Budea <baron at caesar.elte.hu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|5.2 all versions            |4.0.0.3 release
           Keywords|bibisectRequest, regression |implementationError
            Summary|FILEOPEN PPTX: extraneous   |FILEOPEN PPTX: image shadow
                   |line below an image         |that should be invisible
                   |                            |shown as extraneous line
                   |                            |below

--- Comment #5 from Aron Budea <baron at caesar.elte.hu> ---
The line is a shadow on the image, in PowerPoint it can't be seen (probably
because it's configured that way), but in Impress it's shown.

The change could be bisected to the following using repo bibisect-43all (it was
already buggy in oldest commit of bibisect-41max):
https://cgit.freedesktop.org/libreoffice/core/log/?qt=range&q=9351d0e4181924c3f72be24081fc7af027aa41f7..fba5febdf60b37be69d2ffc66445d3e324826346

Found two commits in the range that deal with import of shadows on objects:
https://cgit.freedesktop.org/libreoffice/core/commit/?id=f0efecfb69b336e064e7c8dd2597655eff07944f
https://cgit.freedesktop.org/libreoffice/core/commit/?id=e1a509f4a362d21248b439c99e3b55f4fa9ba1bd

All in all, it seems like an implementation error rather than a regression,
it's just previously the shadow wasn't imported at all (when making the shadow
visible in PP, it still can't be seen when imported in an older LO version).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20200125/338db45b/attachment.htm>