Closed
Bug 39360
Opened 25 years ago
Closed 25 years ago
dynamic table element regression
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
VERIFIED
INVALID
M17
People
(Reporter: rginda, Assigned: waterson)
Details
Attachments
(2 files)
Sometime between today (5/15) and 5/9, tables created dynamically stopped
working correctly.
Dynamically creating a table consisting of one row with two columns appears
instead as one column with two rows.
The attached testcase demonstrates the regression.
Reporter | ||
Comment 1•25 years ago
|
||
Reporter | ||
Comment 2•25 years ago
|
||
Reporter | ||
Comment 3•25 years ago
|
||
Just to clarify...
1) save the xul file as tabletest.xul
2) save the javascript file as tabletest.js
3) start mozilla and browse to file:///<path-to-testcase>/tabletest.xul
expected:
static table
Col 1 Col 2
dynamic table
Col 1 Col 2
actual:
static table
Col 1 Col 2
dynamic table
Col 1
Col 2
Comment 4•25 years ago
|
||
ChrisW, in Viewer after dumping the frames, there is no 2nd table. The content
model suggests that there should be a 2nd table. I'm giving this to you since
you know XUL and want to know more about frame construction.
Assignee: karnaze → waterson
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → M17
Assignee | ||
Comment 5•25 years ago
|
||
rginda: is this related to jst's changes for namespaces? (have those even gone
in yet?) I think your JS should be using createElementNS()...
Comment 6•25 years ago
|
||
Most of my changes are in now except the change in the behaviour of the
properties .nodeName and .tagName, and we should still support creation of
namespace elements with createElement() but DON'T DO THAT, it WILL go away.
The script in the sample is incorrect, it might still work but you should NOT
rely on createElement() knowing about namespace, use:
document.createElementNS("http://www.w3.org/TR/REC-html40", "html:table");
same goes for getElementsByTagName(), use getElementsByTagNameNS().
I haven't had a closer look at this yet so I don't really know what could be
causing it.
Reporter | ||
Comment 7•25 years ago
|
||
That seems to fix it.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•