Closed
Bug 118501
Opened 23 years ago
Closed 16 years ago
vertical-align:top aligns top of box not loose subtree [INLINE]
Categories
(Core :: Layout: Block and Inline, defect, P1)
Core
Layout: Block and Inline
Tracking
()
RESOLVED
FIXED
mozilla1.9.1a1
People
(Reporter: ian, Assigned: dbaron)
References
(Blocks 1 open bug)
Details
(Keywords: css2, testcase, Whiteboard: [Hixie-P2])
Attachments
(3 files, 3 obsolete files)
2.52 KB,
text/html
|
Details | |
3.52 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
14.48 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
STEPS TO REPRODUCE Vertical-align an inline element top, then insert a baseline inline element in the first with a bigger line height. The top of the line box should align with the top of the inner inline element according to dibm. http://mozilla.org/docs/web-developer/quirks/ ACTUAL RESULTS We currently implement the non-sensical CSS2 per-spec behaviour, namely, the outer (top aligned) inline is aligned with the top of the line box, meaning the line box doesn't contain the whole line. See: http://lists.w3.org/Archives/Member/w3c-css-wg/2002JanMar/0021.html Since our current behaviour is technically correct, this is a minor bug.
Reporter | ||
Comment 1•23 years ago
|
||
Er. The first URI above was supposed to be: http://www.people.fas.harvard.edu/~dbaron/css/2000/01/dibm
Reporter | ||
Comment 2•23 years ago
|
||
View the source of the test case for some more explanatory comments with diagrams, showing where the logical line boxes should fall.
Updated•23 years ago
|
Target Milestone: --- → Future
Assignee | ||
Comment 3•23 years ago
|
||
*** Bug 118151 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 4•23 years ago
|
||
Our behaviour recently become wrong. The spec is being changed to match David's description. This bug is therefore now valid.
Reconfirmed using FizzillaCFM/2002071208. Setting All/All.
OS: Linux → All
Hardware: PC → All
Assignee | ||
Comment 6•22 years ago
|
||
*** Bug 168927 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•22 years ago
|
Component: Layout → Layout: Block & Inline
Assignee | ||
Comment 8•20 years ago
|
||
*** Bug 247536 has been marked as a duplicate of this bug. ***
Comment 9•17 years ago
|
||
does bug 364961 comment 5 look to be a dupe?
Assignee | ||
Comment 10•16 years ago
|
||
This removes duplication of the function I need to modify for the real fix (with the addition of 3 lines of code for the VALIGN_OTHER case), and instead just does 2 separate loops.
Assignee | ||
Comment 11•16 years ago
|
||
This is the fix (now that the main part needs to be made in only one place). I still need to write reftest, both for this bug, and for other vertical alignment behavior.
Assignee | ||
Comment 12•16 years ago
|
||
Attachment #321172 -
Attachment is obsolete: true
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Target Milestone: Future → mozilla2.0
Assignee | ||
Comment 13•16 years ago
|
||
Attachment #321175 -
Attachment is obsolete: true
Attachment #322408 -
Flags: superreview?(roc)
Attachment #322408 -
Flags: review?(roc)
Assignee | ||
Comment 14•16 years ago
|
||
Now with a test added that covers a good bit. Firefox before this patch fails exactly the four test items within it that are labeled with this bug number and passes the rest; with the patch the test passes.
Attachment #321173 -
Attachment is obsolete: true
Attachment #322409 -
Flags: superreview?(roc)
Attachment #322409 -
Flags: review?(roc)
Attachment #322408 -
Flags: superreview?(roc)
Attachment #322408 -
Flags: superreview+
Attachment #322408 -
Flags: review?(roc)
Attachment #322408 -
Flags: review+
Attachment #322409 -
Flags: superreview?(roc)
Attachment #322409 -
Flags: superreview+
Attachment #322409 -
Flags: review?(roc)
Attachment #322409 -
Flags: review+
Assignee | ||
Comment 15•16 years ago
|
||
http://hg.mozilla.org/mozilla-central/index.cgi/rev/fd7f8513fa8c http://hg.mozilla.org/mozilla-central/index.cgi/rev/851965dfe20d -> fixed (for 1.9.1)
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: mozilla2.0 → mozilla1.9.1
Comment 16•16 years ago
|
||
<http://mxr.mozilla.org/mozilla-central/source/layout/reftests/box-properties/CSS21-t100801-vertical-align.xhtml> should hopefully work whenever anyone gets around to reading this; it's "not found" right now.
Flags: in-testsuite+
Assignee | ||
Updated•16 years ago
|
Target Milestone: mozilla1.9.1 → mozilla1.9.1a1
You need to log in
before you can comment on or make changes to this bug.
Description
•