The default bug view has changed. See this FAQ.

line-height ignored on (certain) table cells [INLINE]

ASSIGNED
Assigned to

Status

()

Core
Layout: Block and Inline
P2
normal
ASSIGNED
15 years ago
8 years ago

People

(Reporter: Hixie (not reading bugmail), Assigned: dbaron)

Tracking

({css1, css2, testcase})

Trunk
Future
css1, css2, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Hixie-P2][CSS1-5.4.8], URL)

(Reporter)

Description

15 years ago
The line-height property is being ignored on table cells.

   http://www.hixie.ch/tests/adhoc/css/box/inline/007.xml

This is a serious bug, and is one of the few remaining issues we have with our
inline box model.

This is a recent regression.
(Reporter)

Updated

15 years ago
Keywords: css1, css2, regression, testcase
QA Contact: petersen → ian
Whiteboard: [Hixie-P1]
(Assignee)

Comment 1

15 years ago
Ian is referring to bug 151620.
Keywords: css1, css2, regression, testcase
QA Contact: ian → petersen
Whiteboard: [Hixie-P1]
(Assignee)

Updated

15 years ago
Keywords: css1, css2, regression, testcase
QA Contact: petersen → ian
Whiteboard: [Hixie-P1]
(Reporter)

Comment 2

15 years ago
HTML testcase: http://www.hixie.ch/tests/adhoc/css/box/inline/007.html

Comment 3

15 years ago
I was afraid of that after looking at how the patch actually operated.  See bug
152959 comment 52 and bug 152959 comment 53 for an alternate (but not as simple)
proposal on how to fix the issue at hand while avoiding bugs like this.
(Reporter)

Comment 4

15 years ago
Some more testcases:
   http://www.hixie.ch/tests/adhoc/css/box/inline/008.xml
   http://www.hixie.ch/tests/adhoc/css/box/inline/008.html
(Reporter)

Comment 5

15 years ago
Even more testcases:
   http://www.hixie.ch/tests/adhoc/css/box/inline/009.xml
   http://www.hixie.ch/tests/adhoc/css/box/inline/009.html

All six testcases work fine on 1.0 branch builds and on trunk builds from before
the bug 151620 checkin.
(Reporter)

Updated

15 years ago
Depends on: 153032

Updated

15 years ago
Keywords: mozilla1.1

Comment 6

15 years ago
If the fix for bug 151620 was backed out with the landing of bug 153032, isn't
this bug fixed now?
(Reporter)

Comment 7

15 years ago
Strangely, no. the following test case (XML and HTML versions) are still broken:
   http://www.hixie.ch/tests/adhoc/css/box/inline/008.xml
   http://www.hixie.ch/tests/adhoc/css/box/inline/008.html
Assignee: attinasi → dbaron
Severity: major → normal
Summary: line-height ignored on table cells [INLINE] → line-height ignored on (certain) table cells [INLINE]
Whiteboard: [Hixie-P1] → [Hixie-P2]
(Assignee)

Comment 8

15 years ago
Yeah, I did mention at one point in another bug that I thought our behavior
wasn't quite right.
(Assignee)

Comment 9

15 years ago
In any case, not a regression, and no longer critical for 1.1 (I think).
Status: NEW → ASSIGNED
Keywords: mozilla1.1, regression
Priority: -- → P2
Target Milestone: --- → mozilla1.2alpha
(Assignee)

Updated

15 years ago
Target Milestone: mozilla1.2alpha → Future
Whiteboard: [Hixie-P2] → [Hixie-P2][CSS1-5.4.8]
(Assignee)

Updated

15 years ago
Component: Layout → Layout: Block & Inline
QA Contact: ian → layout.block-and-inline
You need to log in before you can comment on or make changes to this bug.