box with background-attachment:fixed goes on top of floats

RESOLVED FIXED in mozilla1.9.2a1

Status

()

defect
RESOLVED FIXED
12 years ago
11 years ago

People

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

Tracking

1.8 Branch
mozilla1.9.2a1
x86
Windows XP
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(1 attachment)

Reporter

Description

12 years ago
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.
Reporter

Updated

12 years ago
Component: Disability Access → General
Reporter

Updated

12 years ago
Version: unspecified → 2.0 Branch
Component: General → Layout
Product: Firefox → Core
QA Contact: disability.access → layout
Version: 2.0 Branch → 1.8 Branch
Assignee

Comment 1

11 years ago
Buggy in the old Fx2, but fixed in Firefox 3 and higher.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WORKSFORME
Assignee

Comment 3

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

Comment 5

11 years ago
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+
Assignee

Updated

11 years ago
Flags: in-testsuite?
Keywords: checkin-needed
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]
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.