Created attachment 178872 [details] The example file and its pptx version in 7.3 When a document containing a default cloud shape is saved as PPTX and reopened, it loses its area fill. 1, Open attachment 125662 [details] from bug 100390 2, Save as PPTX, reopen The blue are fill of the cloud is lost. Started in 7.3 with: https://git.libreoffice.org/core/+/59b16fdf4e25db134ed52457c857eae5241a75e5%5E%21 author Tibor Nagy <nagy.tibor2@nisz.hu> Tue Jan 04 14:38:47 2022 +0100 committer Xisco Fauli <xiscofauli@libreoffice.org> Thu Feb 03 10:04:46 2022 +0100 tdf#137675 PPTX export: adding missing fill="none" to a:path Adding CC to: Tibor Nagy
The problem is deeper than "fill or not fill". The command drawing::EnhancedCustomShapeSegmentCommand::ENDSUBPATH is not evaluated at all. Thus only one <a:path> element exists in <a:pathLst>. But there should be several elements. For each part which ends with "N" in ODF, the export to OOXML has to generate an own <a:path> element.
Created attachment 178888 [details] Example file with simple commands The attached file has the path M 4000 0 L 0 2000 4000 4000 2000 2000 Z S N M 4000 0 L 0 2000 4000 4000 F N Let PowerPoint read the odp file and convert it to pptx. Because the path is really simple, PowerPoint is able to do that. Such way you see in the pptx file what LO needs to export. The F and S commands are applied to the entire subpath. So the first subpath M 4000 0 L 0 2000 4000 4000 2000 2000 Z S N gets a fill but no stroke, and the second subpath M 4000 0 L 0 2000 4000 4000 F N gets a stroke but no fill. It is no regression by the mentioned commit, but handling of ENDSUBPATH was missing before.
I'll try to fix it.
Regina Henschel committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/2029b2f6dd0109c5892e5ac5640022b31fe42fd2 tdf#147978 export subpaths individually in custGeom It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Regina Henschel committed a patch related to this issue. It has been pushed to "libreoffice-7-3": https://git.libreoffice.org/core/commit/6bd85136efe3d3668b59a596d692f65bf0a4982c tdf#147978 export subpaths individually in custGeom It will be available in 7.3.3. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.