Closed Bug 235768 Opened 20 years ago Closed 20 years ago

mozilla 1.6 and up don't always display fixed background correctly. moz 1.4 was fine

Categories

(Core :: Web Painting, defect, P1)

x86
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla1.7beta

People

(Reporter: jbrown, Assigned: bzbarsky)

References

()

Details

Attachments

(2 files)

User-Agent:       
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20040114 Firebird/0.7

On the page http://emergence.uk.net/about/ - you cannot see the smoke and
railtrails through the main content section with moz 1.6. moz 1.4 is fine.

Reproducible: Always
Steps to Reproduce:
1.go to http://emergence.uk.net/about/
2.look at it
3.feel sad

Actual Results:  
i felt more and more sad

Expected Results:  
displayed the background image under the text
The problem here is that fixed-attachment backgrounds should just be placed
relative to the viewport, per CSS2.1, without the "nearest scrolling ancestor"
stuff we do.
Attached patch Maybe like this?Splinter Review
Attached patch Same as diff -wSplinter Review
Comment on attachment 142393 [details] [diff] [review]
Same as diff -w

roc, what do you think?
Attachment #142393 - Flags: superreview?(roc)
Attachment #142393 - Flags: review?(roc)
I think Hixie was planning to change CSS, or something. At least I blame him for
advocating the "nearest scrollframe" behaviour.
Comment on attachment 142393 [details] [diff] [review]
Same as diff -w

This looks like a good patch to make us CSS2.1 compliant.
Attachment #142393 - Flags: superreview?(roc)
Attachment #142393 - Flags: superreview+
Attachment #142393 - Flags: review?(roc)
Attachment #142393 - Flags: review+
Heh.  Well, the nearest scrollframe changed when we changed the impl of
overflow:hidden....

Taking bug.
Assignee: roc → bzbarsky
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P1
Target Milestone: --- → mozilla1.7beta
Checked in.  I also filed bug 235788 on some perf issues I was seeing on that page.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Thanks guys!
(In reply to comment #5)
> I think Hixie was planning to change CSS, or something. At least I blame him 
> for advocating the "nearest scrollframe" behaviour.

That rings a bell. Seems I lost the argument in the WG though. :-)
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: