Closed
Bug 1809
Opened 26 years ago
Closed 25 years ago
<table width=0> should be handled slightly differently
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
VERIFIED
FIXED
M14
People
(Reporter: buster, Assigned: karnaze)
References
Details
(Keywords: testcase, Whiteboard: [TESTCASE] elentar@pacbell.net)
Attachments
(1 file)
(deleted),
text/html
|
Details |
width=0 on a table tag is handled as a special case in layout code in
several places. This gives the right result, but ultimately it is the wrong
approach.
from Steve's original note:
table attribute width=0 is treated as width=auto, not width=min
Peter: do we want to differentiate between HTML width=0 and CSS width=0?
Peter's response:
Yeah, I'd do the fixup in the MapAttributesInto code, converting 0 to auto
there. Then let the layout code respect 0 as 0, then CSS won't be
affected.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Comment 2•26 years ago
|
||
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Assignee | ||
Updated•26 years ago
|
Target Milestone: M7
Assignee | ||
Comment 3•26 years ago
|
||
Moving to M7.
Assignee | ||
Comment 4•26 years ago
|
||
Moving to M9.
Comment 6•26 years ago
|
||
Per bug 5445, width=0% also has the meaning of width=auto under 4.x. Currently,
we are only mapping width=0 as auto. When changing the way that width=0 is
handled, don't forget to make sure that 0% also triggers the 'auto' code.
See also the test case for bug 5445:
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=464
Comment 7•26 years ago
|
||
Updated•26 years ago
|
Whiteboard: [MAKINGTEST] elentar@pacbell.net
Comment 8•26 years ago
|
||
[TESTCASE]
View the attachment (#769) to compare four tables (width=0, width=0%,
CSS/width=0, CSS/width=0%). Currently, width=0 and CSS/width=0 map to "auto" and
width=0% and CSS/width=0% map to "min". At least, that's how it appears.
Both CSS widths should map to "min" and both HTML widths should map to "auto".
Updated•26 years ago
|
Whiteboard: [MAKINGTEST] elentar@pacbell.net → [TESTCASE] elentar@pacbell.net
Assignee | ||
Comment 9•25 years ago
|
||
Moving to M13.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M13 → M14
Assignee | ||
Comment 10•25 years ago
|
||
mass move to m14.
Comment 11•25 years ago
|
||
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
Assignee | ||
Comment 12•25 years ago
|
||
Fixed with latest checkin.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 13•25 years ago
|
||
Testcase looks fine. Based on Karnaze's comments, verifying bug fixed.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•