Closed
Bug 1040
Opened 26 years ago
Closed 26 years ago
CSS vertical-align problems; reflow strange
Categories
(Core :: CSS Parsing and Computation, defect, P2)
Tracking
()
VERIFIED
FIXED
M4
People
(Reporter: dbaron, Assigned: buster)
References
()
Details
vertical-align: middle (bug 606, maybe), vertical-align: text-bottom (I'm not
sure about this one), and vertical-align: 50% do not work.‰
Reporter | ||
Updated•26 years ago
|
Summary: CSS vertical-align → CSS vertical-align problems; reflow strange
Reporter | ||
Comment 1•26 years ago
|
||
Oh... forgot to mention - the reflow of this page is a bit weird as the images
are loading.‰
Updated•26 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•26 years ago
|
||
I think 50% is now OK, and middle is definitely fixed, but now I think text-top
is also wrong. I'm not completely sure, but my current thinking is that text-
top and text-bottom are now the ones wrong.‰
Updated•26 years ago
|
Assignee: peterl → kipp
Status: ASSIGNED → NEW
Comment 3•26 years ago
|
||
The only case that looked wrong to me is "bottom-aligned" (it looks like it's
acting like "text-bottom-aligned"
Reporter | ||
Comment 4•26 years ago
|
||
I think this is all fixed, except that the alignment of the image is a bit
strange in the bottom and the text-bottom align tests. (The image has no given
alignment, so it should be baseline aligned with its parent.) This refers to
the new version of the test suite, which should be at that above URL soon --
the test is a bit different.
This test now passes gloriously. The problems were: handling of margins around
inline elements when vertically aligning; handling of the line-height property
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 7•26 years ago
|
||
Verified fixed. There could be some 1px error issues, but I'll address them in
bug 1990, which has a better test for that.
You need to log in
before you can comment on or make changes to this bug.
Description
•