Tables print backgrounds.

VERIFIED FIXED in mozilla0.9.7

Status

()

VERIFIED FIXED
17 years ago
15 years ago

People

(Reporter: dcone, Assigned: dcone)

Tracking

({topembed})

Trunk
mozilla0.9.7
x86
Windows NT
topembed
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

17 years ago
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.
(Assignee)

Updated

17 years ago
Keywords: topembed
(Assignee)

Comment 1

17 years ago
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 2

17 years ago
r=rods

Comment 3

17 years ago
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?
Target Milestone: --- → mozilla0.9.7
(Assignee)

Comment 5

17 years ago
That code should print a darker text on white.. just for now until I fix bug 
96870.  If you did that now.. with dark text on a background.. it would print 
dark on dark since the text is automatically darkened for body backgrounds.  
This is all addressed and will be fixed in bug 96870.
(Assignee)

Comment 6

17 years ago
Fixed on trunk
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Keywords: edt0.9.4
Resolution: --- → FIXED

Comment 7

17 years ago
Sujay -- This needs to be verified on the trunk before we can plus (+) for 094

Comment 8

17 years ago
what are the test cases we should excersize to verfy this one?

Comment 9

17 years ago
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.

Comment 10

17 years ago
Is 2001-11-26-08-trunk sufficient for verification ?

Comment 11

17 years ago
actually this was fixed on 11/27, so we will need 11/27 and later build
to verify on.

Comment 12

17 years ago
verified in 11/27 build.
Status: RESOLVED → VERIFIED

Comment 13

17 years ago
based on the trunk verification, I'm adding + to edt0.9.4
Keywords: edt0.9.4 → edt0.9.4+

Comment 14

17 years ago
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.

Comment 15

17 years ago
Created attachment 59752 [details]
background on table

Comment 16

17 years ago
Created attachment 59753 [details]
background on table cell
EDT - Let's get this on the branch tonite if we can - Embedding friends are
pulling tomorrow. 
(Assignee)

Comment 18

17 years ago
checked into the branch
(Assignee)

Updated

17 years ago
Keywords: fixed0.9.4

Comment 19

17 years ago
verified in 1/13 branch build.
Keywords: verified0.9.4

Updated

15 years ago
Keywords: fixed0.9.4
You need to log in before you can comment on or make changes to this bug.