Closed
Bug 124478
(paintperf1)
Opened 23 years ago
Closed 17 years ago
remove paint layers
Categories
(Core :: Layout: Floats, defect, P2)
Core
Layout: Floats
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: dbaron, Assigned: dbaron)
References
Details
(Keywords: perf, Whiteboard: [whitebox])
The paint layers are badly named. We should rename
NS_FRAME_PAINT_LAYER_BACKGROUND to NS_FRAME_PAINT_LAYER_BLOCK and rename
NS_FRAME_PAINT_LAYER_FOREGROUND to NS_FRAME_PAINT_LAYER_INLINE. (If we don't
get rid of them completely first.)
Updated•23 years ago
|
Priority: -- → P4
Assignee | ||
Comment 1•23 years ago
|
||
I'm going to turn this into a bug on removing paint layers, which should speed
up painting a bit, and should (I hope) be allowed by the next version of CSS.
Status: NEW → ASSIGNED
Keywords: perf
Priority: P4 → P2
Summary: rename paint layers → remove paint layers
Target Milestone: --- → mozilla1.1beta
Assignee | ||
Updated•22 years ago
|
Target Milestone: mozilla1.1beta → Future
Updated•22 years ago
|
Alias: paintperf1
Assignee | ||
Updated•22 years ago
|
Component: Layout → Layout: Floats
Comment 2•22 years ago
|
||
Ugh. Do we really want floats to paint on top of text?
QA Contact: petersen → ian
Assignee | ||
Comment 3•22 years ago
|
||
Why not? It only matters when people use negative margins, negative text
indent, or something similar.
BTW we need to change things to create a view for every float (because floats
can have a containing block other than their parent element, and this needs to
be reflected into the view system to fix some obscure bugs). This would also
have an effect on z-ordering.
Assignee | ||
Comment 5•22 years ago
|
||
Why do floats need views? According to CSS2, they *can't* have views because
that would break the z-ordering that paint layers do, but that's probably
relaxed in CSS 2.1.
Assignee | ||
Comment 6•22 years ago
|
||
(BTW, I think I want to start attaching floats to their containing block rather
than their closest block, which they have very little to do with. That might
need to happen first. Is that what you're talking about?)
There's already a pile of logic in nsCSSFrameConstructor to attach floats to the
floater containing block. Does that not work?
Assignee | ||
Comment 8•22 years ago
|
||
I think they go in the floater list of the nearest block, which is technically
the containing block according to the spec, but not logically.
So the spec is wrong and will be changed?
Also, like all other frames, there are various reasons why we might want to
attach a view to a float, for example if it's translucent. We have to be able to
do that successfully.
Comment 10•22 years ago
|
||
This seems to make things (much) easier for bug 185977
Comment 11•22 years ago
|
||
Note that even if floats paint at the z-index:0 layer, or some other layer on
top of blocks, all inlines should still paint on top of all blocks backgrounds
and borders, according to all the proposals I've seen sent to the WG.
Personally I am still attached to the idea of having floats paint over other
blocks, but having all inline elements paint above everything in normal flow.
Updated•22 years ago
|
Whiteboard: [whitebox]
What about inlines that contain blocks?
Can't we just continue to say "document order", possibly clarify that by saying
that if A contains B then A comes before B in the document, and then be done
with it? That will guarantee inlines on top of the blocks that contain them.
Assignee | ||
Comment 13•17 years ago
|
||
I think the spec is pretty settled here, so let's go with wontfix. (The layers still exist as part of nsDisplayList, but with new names.)
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•