Open
Bug 1309830
Opened 8 years ago
Updated 2 years ago
nsBlockFrame::SplitLine() does not redo current line when its block-size shrinks
Categories
(Core :: Layout: Floats, defect, P3)
Core
Layout: Floats
Tracking
()
REOPENED
People
(Reporter: TYLin, Unassigned)
References
(Blocks 2 open bugs)
Details
I had attempted to fix 345369-2.html [1] in bug 1291110 comment 17, but I failed to find a feasible solution.
The issue is that when nsBlockFrame::SplitLine() split some frames to its own line during reflow, the height of original line could become shorter due to those frames being split. Therefore it's possible that the original line won't intersect some of the floats now, and its inline-start position could be adjusted. Currently, we do no redo the original line.
[1] https://dxr.mozilla.org/mozilla-central/rev/22be4ae74653b25186665f22e52a50e7027fd36b/layout/reftests/floats/reftest.list#14
Updated•8 years ago
|
Blocks: shape-outside, 1309467
Updated•8 years ago
|
Priority: -- → P3
Comment 1•6 years ago
|
||
Bug 1466231 is handling this, perhaps in a different way than Ting-Yu diagnosed originally.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Comment 2•6 years ago
|
||
The approach for Bug 1466231 has changed enough that it no longer resolves this problem. I'll take this bug since the solution will likely be closely related.
Assignee: nobody → bwerth
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 3•6 years ago
|
||
Taking myself off this again to focus on other work. Sorry for the thrash.
Assignee: bwerth → nobody
Comment 4•6 years ago
|
||
This is the only bug left blocking the shape-outside bug 1098939, which itself is the only blocking bug for the CSS Shapes Module Level 1 (bug 1040714).
Would be great to see this fixed, so the other both can be closed, too! (And finishing another CSS module would be fantastic progress, since we just have two completed so far (see bug 1323667).
Sebastian
Comment 5•6 years ago
|
||
Ting-Yu, do you want to do more shape-outside work and take this bug? If not, I'll take it back and try again.
Flags: needinfo?(aethanyc)
Reporter | ||
Comment 6•6 years ago
|
||
Brad, I'm currently busy with Bug 616436. I would be great if you can take this. Thanks!
Flags: needinfo?(aethanyc)
Updated•6 years ago
|
Assignee: nobody → bwerth
Updated•4 years ago
|
Assignee: bwerth → nobody
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•