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)

1.8 Branch
x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: qi.jiahui, Assigned: crazy-daniel)

References

()

Details

Attachments

(1 file)

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.
Component: Disability Access → General
Version: unspecified → 2.0 Branch
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
Probably fixed by bug 317375.
Depends on: 317375
Attached patch reftest [Checkin: Comment 7] (deleted) — Splinter Review
A slightly simplified version of the reporter's testcase.
Attachment #355064 - Flags: superreview?(dbaron)
Attachment #355064 - Flags: review?(dbaron)
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 on attachment 355064 [details] [diff] [review] reftest [Checkin: Comment 7] r+sr=dbaron
Attachment #355064 - Flags: superreview?(dbaron)
Attachment #355064 - Flags: superreview+
Attachment #355064 - Flags: review?(dbaron)
Attachment #355064 - Flags: review+
Flags: in-testsuite?
Keywords: checkin-needed
Attachment #355064 - Attachment description: reftest → reftest [Checkin: Comment 7]
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.

Attachment

General

Creator:
Created:
Updated:
Size: