Bug 154853 - FILESAVE PPTX Order of radial gradients steps is reversed
Summary: FILESAVE PPTX Order of radial gradients steps is reversed
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.6.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Object-Fill-Gradient
  Show dependency treegraph
 
Reported: 2023-04-17 07:57 UTC by Gabor Kelemen (allotropia)
Modified: 2023-04-21 21:11 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file from PP with 5 color steps (30.92 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2023-04-17 07:57 UTC, Gabor Kelemen (allotropia)
Details
The example file saved by LO 76 (23.87 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2023-04-17 07:58 UTC, Gabor Kelemen (allotropia)
Details
The original and its LO-saved version in PP (249.65 KB, image/png)
2023-04-17 07:59 UTC, Gabor Kelemen (allotropia)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Kelemen (allotropia) 2023-04-17 07:57:19 UTC
Created attachment 186708 [details]
Example file from PP with 5 color steps

Attached file contains a circle shape with multi color gradients.

Opening this is now supported in master with MCGR_TEST=0 environment. The 5 color steps are imported correctly.
Saving this however reverses their order.

1. Open attached file with MCGR_TEST=0 instdir/program/soffice
2. Save as PPTX, reopen it in PP

-> The blue and white color steps and the green and yellow ones swap places.

Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 19ecb4a390bc0bc6dad6688cdc2b70a8d05bcde2
CPU threads: 14; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: hu-HU (hu_HU); UI: en-US
Calc: threaded
Comment 1 Gabor Kelemen (allotropia) 2023-04-17 07:58:36 UTC
Created attachment 186709 [details]
The example file saved by LO 76

This also has the patch https://gerrit.libreoffice.org/c/core/+/150416 so that colors are visible in the canvas.
Comment 2 Gabor Kelemen (allotropia) 2023-04-17 07:59:00 UTC
Created attachment 186710 [details]
The original and its LO-saved version in PP
Comment 3 Julien Nabet 2023-04-17 10:18:38 UTC
Just for the record, on pc Debian x86-64 with master sources updated today, when reopening the saved file with LO, I just go a white circle with a blue border.
Comment 4 Gabor Kelemen (allotropia) 2023-04-21 21:11:55 UTC
This was fixed by (expanding the gerrit linked above):

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

author	Armin Le Grand (allotropia) <armin.le.grand.extern@allotropia.de>	2023-04-14 16:29:18 +0200
committer	Armin Le Grand <Armin.Le.Grand@me.com>	2023-04-17 16:10:48 +0200
commit dbbe5d0cd721a815df5e4cbf3215f291a423f2b1 (patch)

MCGR: 1st corrections to gradient export, transparency

Changed Alpha export from using Red component of
used BColor too use luminance, that will be more
safe if we evtl use same gradients for this in the
future.

Added evtl. needed inversion for gradient exports,
also emulation of our 'axial' type.

thx Armin :)