Closed
Bug 389623
Opened 17 years ago
Closed 16 years ago
box with background-attachment:fixed goes on top of floats
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.2a1
People
(Reporter: qi.jiahui, Assigned: crazy-daniel)
References
()
Details
Attachments
(1 file)
(deleted),
patch
|
dbaron
:
review+
dbaron
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5) Gecko/20070713 Firefox/2.0.0.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5) Gecko/20070713 Firefox/2.0.0.5
According to CSS2.1, floats should overlap other non-positioned boxes in the normal flow, including it's backgrounds and borders. In the test case above, the div.fixed_bg got a background image and it's background-attachment was set to fixed, this resulted to the div to go on top of the float.
Reproducible: Always
Adding position:relative to the floats would cure this bug.
Updated•17 years ago
|
Component: General → Layout
Product: Firefox → Core
QA Contact: disability.access → layout
Version: 2.0 Branch → 1.8 Branch
Buggy in the old Fx2, but fixed in Firefox 3 and higher.
Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → WORKSFORME
A slightly simplified version of the reporter's testcase.
Attachment #355064 -
Flags: superreview?(dbaron)
Attachment #355064 -
Flags: review?(dbaron)
Comment 4•16 years ago
|
||
Did you test that the reftest failed in a build before the fix and passed afterwards? (You can test just by loading the pages in two tabs and switching between the tabs...)
I'm not sure if I understand you correctly.
If you ask wether I've tested if bug 317375 was causing this, then no, I didn't test for this.
If you're asking wether I've tested both, testcase and reference, in a buggy build and in a trunk build as well, then yes, I did (I always do).
Comment 6•16 years ago
|
||
Attachment #355064 -
Flags: superreview?(dbaron)
Attachment #355064 -
Flags: superreview+
Attachment #355064 -
Flags: review?(dbaron)
Attachment #355064 -
Flags: review+
Flags: in-testsuite?
Keywords: checkin-needed
Comment 7•16 years ago
|
||
Comment on attachment 355064 [details] [diff] [review]
reftest
[Checkin: Comment 7]
http://hg.mozilla.org/mozilla-central/rev/6ece7e3224ca
Attachment #355064 -
Attachment description: reftest → reftest
[Checkin: Comment 7]
Comment 8•16 years ago
|
||
Re-set checkin-needed (with whiteboard comment) if you want 1.9.1/... too.
Assignee: nobody → crazy-daniel
Flags: in-testsuite? → in-testsuite+
Keywords: checkin-needed
Resolution: WORKSFORME → FIXED
Target Milestone: --- → mozilla1.9.2a1
You need to log in
before you can comment on or make changes to this bug.
Description
•