Summary: | Impossible in 7.4 to export a PNG with a transparent background (no background defaults to white) | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | xordevoreaux |
Component: | Draw | Assignee: | Armin Le Grand <Armin.Le.Grand> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | 79045_79045, Armin.Le.Grand, ilmari.lauhakangas, telesto, xiscofauli, xordevoreaux |
Priority: | medium | Keywords: | bibisected, bisected, regression |
Version: | 7.4.0.0 alpha0+ | ||
Hardware: | x86-64 (AMD64) | ||
OS: | All | ||
See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=123973 | ||
Whiteboard: | target:7.4.0 target:7.3.1 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 109323 | ||
Attachments: |
Screen capture of Sims 4 Studio with properly exported PNG
Screen capture of Sims 4 Studio with PNG with no transparency Proper PNG export with transparency PNG export from 7.4 where transparency defaulted to white LibreOffice Draw file |
Description
xordevoreaux
2022-02-02 05:35:46 UTC
Created attachment 177973 [details]
Screen capture of Sims 4 Studio with properly exported PNG
This image shows clothing on a 3D mesh of a person. Notice that skin is showing on the person. The area where there is no clothing but instead shows skin is where I have transparency set on the exported PNG (nothing should block the skin).
Created attachment 177974 [details]
Screen capture of Sims 4 Studio with PNG with no transparency
This is the result of importing a PNG using 7.4 where transparency defaulted to white. Completely useless, as it renders the 3D model white where there should be skin.
Created attachment 177975 [details]
Proper PNG export with transparency
This is the default texture from Sims 4 studio containing the necessary transparency (load it in Windows and you'll see the checkerboard pattern around the clothing where nothing at all is rendering, not even white).
Created attachment 177976 [details]
PNG export from 7.4 where transparency defaulted to white
This is the default image from Sims 4 Studio imported into LO Draw and exported as a PNG with transparency, except... no transparency. If you view the file in Windows, you'll see white rendered around the clothing rather than check boxes indicating alpha transparency.
Not useful.
Created attachment 177977 [details]
LibreOffice Draw file
This is the LibreOffice draw file that I used in 7.4 to export the PNG using transparency. Notice the background slide color is set to None and that View, Master, Background setting is also None, yet exporting the page as a PNG and selecting the transparency option during export renders the background white, not transparent.
Checked with linux-64-7.4 bibisect repo and indeed the cause is https://git.libreoffice.org/core/commit/87866def6bfe2ee91be34a5ce37b79d6da881617 So the fix for tdf#123973 had unexpected side-effects. I will revert that one ASAP and will have to fix it differently - now having another case to check hat may break here - what is a good finding of unexpected internal connectivity. (In reply to Buovjaga from comment #6) > Checked with linux-64-7.4 bibisect repo and indeed the cause is > https://git.libreoffice.org/core/commit/ > 87866def6bfe2ee91be34a5ce37b79d6da881617 Hi, do you know if there is a simple way to revert those changes from tdf#123973, including the backports? Is it necessary to do all solo or is there some mechanism I do not know to do that (half-)automated...? (In reply to Armin Le Grand from comment #8) > (In reply to Buovjaga from comment #6) > > Checked with linux-64-7.4 bibisect repo and indeed the cause is > > https://git.libreoffice.org/core/commit/ > > 87866def6bfe2ee91be34a5ce37b79d6da881617 > > Hi, do you know if there is a simple way to revert those changes from > tdf#123973, including the backports? Is it necessary to do all solo or is > there some mechanism I do not know to do that (half-)automated...? I guess first revert Xisco's test, then in a separate revert your own patch and backport the revert to 7-3 branch. Hi Armin, I reverted it in https://gerrit.libreoffice.org/c/core/+/129351 But I let the decision to push it to you, maybe you want to fix it instead... (In reply to Xisco Faulí from comment #10) > Hi Armin, > I reverted it in https://gerrit.libreoffice.org/c/core/+/129351 But I let > the decision to push it to you, maybe you want to fix it instead... Unittest added in https://gerrit.libreoffice.org/c/core/+/129354 (In reply to Xisco Faulí from comment #11) > (In reply to Xisco Faulí from comment #10) > > Hi Armin, > > I reverted it in https://gerrit.libreoffice.org/c/core/+/129351 But I let > > the decision to push it to you, maybe you want to fix it instead... > > Unittest added in https://gerrit.libreoffice.org/c/core/+/129354 Guys, you make my head spin - could we - someone - just revert the change, maybe the test, too, I do not know if that is needed...? Jus tlet's go to the state before that change, I need to fix that definitely differently... (In reply to Armin Le Grand from comment #12) > (In reply to Xisco Faulí from comment #11) > > (In reply to Xisco Faulí from comment #10) > > > Hi Armin, > > > I reverted it in https://gerrit.libreoffice.org/c/core/+/129351 But I let > > > the decision to push it to you, maybe you want to fix it instead... > > > > Unittest added in https://gerrit.libreoffice.org/c/core/+/129354 > > Guys, you make my head spin - could we - someone - just revert the change, > maybe the test, too, I do not know if that is needed...? > Jus tlet's go to the state before that change, I need to fix that definitely > differently... Hi Armin, Then please give a +1 to https://gerrit.libreoffice.org/c/core/+/129351 (In reply to Xisco Faulí from comment #13) > Hi Armin, > Then please give a +1 to https://gerrit.libreoffice.org/c/core/+/129351 A separate revert has to be created for 7.3.1 (without the test) Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/028602a7fc0691958ade01c9bc7cd617e85203ac tdf#147119:Revert "tdf#123973 fix missing MasterPage content in export" 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. Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/789e22f7eb115d56351f19ec238a5c92798b0a4c tdf#147119: sd_png_export_tests: Add unittest 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. Xisco Fauli committed a patch related to this issue. It has been pushed to "libreoffice-7-3": https://git.libreoffice.org/core/commit/7d9851c65c33865b995932e0d83cee815edcbbce tdf#147119:Revert "tdf#123973 fix missing MasterPage content in export" It will be available in 7.3.1. 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. |