[Libreoffice-bugs] [Bug 141127] New: FILEOPEN ODP LibreOffice uses wrong default skew angle in extruded custom shape

[bugzilla-daemon at bugs.documentfoundation.org]

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

            Bug ID: 141127
           Summary: FILEOPEN ODP LibreOffice uses wrong default skew angle
                    in extruded custom shape
           Product: LibreOffice
           Version: 7.2.0.0.alpha0+ Master
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: medium
         Component: Impress
          Assignee: libreoffice-bugs at lists.freedesktop.org
          Reporter: rb.henschel at t-online.de

Created attachment 170589
  --> https://bugs.documentfoundation.org/attachment.cgi?id=170589&action=edit
Converted from ppt to odp by PowerPoint

Open attached file. It has an extruded shape top/right and another bottom/left.

The top/right one should have direction "North East", but it has direction
"South West".

The file was converted by PowerPoint from a ppt file to odp. The direction of a
parallel projection is specified by the attribute draw:extrusion-skew (§19.177,
ODF 1.3). The second value in it is the angle.
https://docs.oasis-open.org/office/OpenDocument/v1.3/cs02/part3-schema/OpenDocument-v1.3-cs02-part3-schema.html#__RefHeading__1417044_253892949

If the attribute is missing, an angle of 45deg has to be used as default, but
LibreOffice uses -135. The default value is set in GetSkew() in
https://opengrok.libreoffice.org/xref/core/svx/source/customshapes/EnhancedCustomShape3d.cxx?r=cfff893b#85

Unfortunately the same part is used when getting a ppt-file. And there, the
default value is -135deg. So simple changing it there to 45deg, will break ppt
import.

PowerPoint drops the draw:extrusion-skew attribute on saving, if it has default
values. (What corresponds to the idea of "default" values.)

My idea is, not to change the value in GetSkew() and then try to tweak MS
binary import and export, but to add the values on import of odp in 
XMLEnhancedCustomShapeContext::endFastElement() in case the attribute is
missing.

Do yo agree?

-- 
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/20210320/3625058a/attachment.htm>