After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 779160 - Chart objects transparency is not exported to xlsx
Chart objects transparency is not exported to xlsx
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export MS Excel (tm)
1.12.x
Other Linux
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2017-02-24 06:35 UTC by Jean Bréfort
Modified: 2017-02-28 16:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Export this file to xlsx and reopen, the transparency is lost, at least in gnumeric. (14.39 KB, application/x-gnumeric)
2017-02-24 06:35 UTC, Jean Bréfort
  Details
Proposed patch (643 bytes, patch)
2017-02-25 17:52 UTC, Jean Bréfort
none Details | Review
patch to test (611 bytes, patch)
2017-02-28 16:38 UTC, Jean Bréfort
none Details | Review

Description Jean Bréfort 2017-02-24 06:35:36 UTC
Created attachment 346613 [details]
Export this file to xlsx and reopen, the transparency is lost, at least in gnumeric.

When exporting a chart with tansparent objects inside, transparency is lost (except for backplane and legend where the must_fill_fill field is forced to TRUE).
Comment 1 Morten Welinder 2017-02-25 16:28:52 UTC
Confirmed.  Excel renders what we write as filled.
Comment 2 Jean Bréfort 2017-02-25 17:52:45 UTC
Created attachment 346713 [details] [review]
Proposed patch

At least the transparency is preserved in gnumeric and LO (but in LO the outline becomes transparent)
Comment 3 Morten Welinder 2017-02-25 23:53:44 UTC
Hmm...  I was sure that test was there for a reason, but the test suite
disagrees: I see no failures by applying the patch.

I am a strong believer of the test suite, so go ahead and commit.  We don't
break anything we know about.  We ought to add a test for transparent fill,
though.
Comment 4 Jean Bréfort 2017-02-26 08:07:15 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Once that release is available, you may want to check for a software upgrade provided by your Linux distribution.
Comment 5 Morten Welinder 2017-02-26 17:03:15 UTC
Missing commit or push?
Comment 6 Jean Bréfort 2017-02-26 17:12:10 UTC
Previous push apparently failed, not sure why. Done.
Comment 7 Morten Welinder 2017-02-28 16:08:25 UTC
Hmm...  I am not sure why I did not see problems at home, but here I see
lots of changes in t6516 for xlsx:

-                    <fill type="none" auto-type="1" is-auto="1" auto-fore="1"/>
+                    <fill type="none" auto-type="0" is-auto="1" auto-fore="1"/>

I.e., we have lost the ability to round-trip an automatic type.
Comment 8 Jean Bréfort 2017-02-28 16:38:09 UTC
Created attachment 346917 [details] [review]
patch to test

Please, try this.
Comment 9 Morten Welinder 2017-02-28 16:48:46 UTC
That seems to work.