Closed
Bug 75250
Opened 24 years ago
Closed 24 years ago
empty table width attribute treated as zero table width
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: jimw-mo, Assigned: bernd_mozilla)
References
()
Details
Attachments
(4 files)
|
133 bytes,
text/html
|
Details | |
|
726 bytes,
patch
|
Details | Diff | Splinter Review | |
|
1.16 KB,
text/html
|
Details | |
|
759 bytes,
patch
|
Details | Diff | Splinter Review |
with the following html, the table is rendered as if a zero table width were
specified. (http://doc.weblogs.com/ is the site where i noticed this issue.) i
won't pretend to be enough of a standards wonk to say whether this is strictly
wrong, but it appears to be inconsistent with netscape 4 and internet explorer.
(if you remove the 'width=""' from the table element, the text renders on one
line. with it there, it only renders as wide as the widest word.)
<html>
<head>
<title>test</title>
</head>
<body>
<table width="">
<tr><td>this is a simple test.</td></tr>
</table>
</body>
</html>
|
||
Comment 1•24 years ago
|
||
over to parser..
Assignee: karnaze → harishd
Component: Layout → Parser
QA Contact: petersen → bsharma
This ain't parser. Reassigning to karnaze.
Assignee: harishd → karnaze
Verified on:
build: 2001-05-21-11-Mtrunk
platform: WinNT
As this is a HTML table problem, assigning QA to AMAR.
QA Contact: bsharma → amar
|
||
Comment 5•24 years ago
|
||
If it's verified, let's mark it new. ^_^
Status: UNCONFIRMED → NEW
Ever confirmed: true
*** Bug 89739 has been marked as a duplicate of this bug. ***
I have a patch for this, so steeling the bug
Assignee: karnaze → bernd.mielke
|
||
Comment 10•24 years ago
|
||
When fixing this patch, please check the testcases attached to the duped bug 88243.
|
Assignee | |
Comment 11•24 years ago
|
||
Thanks Jacek, the patch now fixes the width="" issue but the width=50" still
remains.
|
|
Assignee | |
Comment 12•24 years ago
|
||
I removed the wrong duplicate as the remaining issue is a parser problem. So the
patch fixes all relevant testcases, for the remaining problem please look at bug
88243.
|
||
Comment 13•24 years ago
|
||
|
|
||
Comment 14•24 years ago
|
||
Changing to correct component and setting platform/os to All, nominating for
mozilla0.9.3
|
||
Comment 15•24 years ago
|
||
Patch looks fine. I might suggest you change the nested IF into a single
compound IF, as in
if ((NS_ERROR_ILLEGAL_VALUE == ec) && (tmp.Length > 0))
but that is just a suggestion, not very important.
sr=attinasi for patch 41539
|
|
Assignee | |
Comment 16•24 years ago
|
||
|
||
Comment 17•24 years ago
|
||
r=karnaze
|
|
Assignee | |
Comment 18•24 years ago
|
||
fix checked in
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 19•24 years ago
|
||
*** Bug 60919 has been marked as a duplicate of this bug. ***
|
||
Comment 20•24 years ago
|
||
In which build can I see this bug fixed? 2001071104 - all the same for me.
|
|
||
Comment 21•24 years ago
|
||
My 2001071104 (WindowsME) fixes it. Have you installed the trunk build (right)
or 0.9.2 build (wrong)? It has not yet been checked in the 0.9.2 branch.
|
|
||
Comment 22•24 years ago
|
||
Trunk build.
bug 88243 is not fixed with this.
http://www.chip-online.com/products_tests/sp_products_tests_4047.html
http://forum.spylog.ru/
http://www.virusbtn.com/100/vb100sum.html
|
|
Assignee | |
Comment 23•24 years ago
|
||
Eugene, no wonder that bug88243 is not fixed, I removed the wrong duplicate
relationship to this bug, both bugs are similiar, but the fixes are different.
SPAM. HTML Element component deprecated, changing component to Layout. See bug
88132 for details.
Component: HTML Element → Layout
You need to log in
before you can comment on or make changes to this bug.
Description
•