Currently tables print backgrounds, this is not good until we have options for the user to set when and how to print backgrounds. So this has to be disabled for printing until bug 96870 is fixed.
Created attachment 59164 [details] [diff] [review] patch to not do backgrounds for printing Patch to turn off backgrounds for printing. This does not effect general backgrounds since this is not called for the body or html background.. but this will turn off table backgrounds.
Comment on attachment 59164 [details] [diff] [review] patch to not do backgrounds for printing sr=attinasi
Attachment #59164 - Flags: superreview+
Wait a sec. So if I have the following: <body style="background-color: white"> <table style="color: white; background-color: black"> </table> </body> What happens with this patch? Does it print white on white? Black on white?
Fixed on trunk
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Sujay -- This needs to be verified on the trunk before we can plus (+) for 094
what are the test cases we should excersize to verfy this one?
Roland/Boris, I don't have testcases to verify this and my PC is in for service. can one of you verify that this is fixed on latest trunk builds and mark this VERIFIED-FIXED? thanks.
Is 2001-11-26-08-trunk sufficient for verification ?
actually this was fixed on 11/27, so we will need 11/27 and later build to verify on.
verified in 11/27 build.
Status: RESOLVED → VERIFIED
based on the trunk verification, I'm adding + to edt0.9.4
Keywords: edt0.9.4 → edt0.9.4+
Attaching two testcases for further verification if needed. First is testcase with table background; second is testcase with cell background. Verified fixed on 11_29_09_trunk build.
EDT - Let's get this on the branch tonite if we can - Embedding friends are pulling tomorrow.
checked into the branch
verified in 1/13 branch build.
You need to log in before you can comment on or make changes to this bug.