Closed
Bug 40546
Opened 25 years ago
Closed 24 years ago
Background color of HTML mail not shown
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: bugzilla, Assigned: pierre)
References
Details
(Whiteboard: [fix in hand][nsbeta1+])
Attachments
(3 files)
687 bytes,
text/html
|
Details | |
916 bytes,
patch
|
Details | Diff | Splinter Review | |
944 bytes,
patch
|
Details | Diff | Splinter Review |
If you send a "HTML mail" where you change the background color of the mail the
backgroud color is not shown in Mozilla.
Build 2000052420
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → M19
Reporter | ||
Comment 1•25 years ago
|
||
This seems fixed...?
Have you done anything? I can see anything checked in!
Comment 2•25 years ago
|
||
Hi Henrik,
Nope...nothing on my end. What you have to realize is that often times, the
bugs you file against mime/me are not me at all, but rather changes in layout.
Just like when vCards break every few weeks. I haven't touched that code in
ages, but it seems to find a way to break and get fixed because layout is
changing constantly.
- rhp
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 3•25 years ago
|
||
Sorry if I file bugs in the wrong components. It's hard to find out whether a
rendering bug is to be filed in the mail&news department or in the browser
department.
But anyways... more time for you to work on other bugs...:) Later!
Comment 5•25 years ago
|
||
This problem is still present even in a recent windows build (2000113004).
This is the original submission of bug 54455.
To see this bug:
Create new message
Change background color
Save as draft or Send to yourself
Open saved or received message
bgcolor is ignored and is shown as white
Message source shows correct color eg <body bgcolor="#009900" ... should display
green background.
Status: VERIFIED → REOPENED
Resolution: WORKSFORME → ---
Comment 8•25 years ago
|
||
marking nsbeta1+ and moving to mozilla0.8
Comment 9•25 years ago
|
||
movign to mozilla0.9 and reassigning to ducarroz
Priority: P2 → P3
Target Milestone: mozilla0.8 → mozilla0.9
Comment 10•25 years ago
|
||
really reassigning this time.
Assignee: rhp → ducarroz
Status: REOPENED → NEW
Comment 11•25 years ago
|
||
*** Bug 54634 has been marked as a duplicate of this bug. ***
Comment 12•25 years ago
|
||
*** Bug 68282 has been marked as a duplicate of this bug. ***
Comment 13•24 years ago
|
||
*** Bug 54455 has been marked as a duplicate of this bug. ***
Comment 14•24 years ago
|
||
*** Bug 70741 has been marked as a duplicate of this bug. ***
Is 6 DUPS good enough to mark mostfreq? If so, someone please mark. Thanks.
Comment 17•24 years ago
|
||
This seems fixed in 2001-03-16-04 / Win98.
Has anyone done anything for this? Thanks!
Comment 18•24 years ago
|
||
I still see the problem on WinNT 2001032104
Comment 19•24 years ago
|
||
*** Bug 73998 has been marked as a duplicate of this bug. ***
Comment 20•24 years ago
|
||
I think it's a layout bug. I will attach a test file that correctly set the
background color when viewed with 4.x or even IE. However, it doesn't with
Mozilla. The file is a capture of the output generated by mime during message
display.
Comment 21•24 years ago
|
||
Comment 22•24 years ago
|
||
Just click on the attachment link above, the backround should be purple.
Reassign to layout...
Assignee: ducarroz → karnaze
Component: MIME → Layout
Keywords: mailtrack
Product: MailNews → Browser
QA Contact: laurel → petersen
Comment 23•24 years ago
|
||
Reassigning to pierre. The external style sheet seems to be causing it. The 2nd
body may not be relevant.
Assignee: karnaze → pierre
Component: Layout → Style System
Target Milestone: mozilla0.9.1 → mozilla1.0
Comment 24•24 years ago
|
||
*** Bug 92318 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 25•24 years ago
|
||
Assignee | ||
Comment 26•24 years ago
|
||
The problem happens because the style declarations in the linked stylesheet have
a higher specificity than the html attributes, and not just because the
stylesheet is declared 'important'. Setting the default color on the html element
instead of the body should fix it.
jeff & daniel: please review
Status: NEW → ASSIGNED
Whiteboard: [nsbeta1+] → [fix in hand][nsbeta1+]
Comment 27•24 years ago
|
||
Patch works great. Thanks for fixing it. R=ducarroz
the BodyFixupRule is going to kill us all ;-) r=glazman
Isn't the selector wrong in the patch for the modern skin, since there's a
default namespace that you need to override? ("html" should be "html|html".)
And might you be better off not specifying anything? After all, with this,
couldn't you end up with a border of one color around a message of the other?
What do we do if there's nothing specified? (Do we use the defaults for the
canvas?)
Assignee | ||
Comment 30•24 years ago
|
||
You are correct for the default namespace. For the background color, my take is
that we want to edit the mail with the background color that is generally used by
emailers, which is white.
Assignee | ||
Comment 31•24 years ago
|
||
Shoot, I still have these changes in my tree and now we need approvals to
checkin...
Marc: please s/r
DBaron: please approve
Target Milestone: mozilla1.0 → mozilla0.9.5
Assignee | ||
Comment 32•24 years ago
|
||
Comment 33•24 years ago
|
||
[s]r=attinasi
Comment 34•24 years ago
|
||
a=asa on behalf of drivers.
Assignee | ||
Comment 35•24 years ago
|
||
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 24 years ago
Resolution: --- → FIXED
Target Milestone: mozilla0.9.5 → mozilla0.9.4
Comment 36•24 years ago
|
||
Marking verified in the Sept 05th build (2001-09-05-05)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•