Closed Bug 39360 Opened 25 years ago Closed 25 years ago

dynamic table element regression

Categories

(Core :: Layout: Tables, defect, P3)

x86
All
defect

Tracking

()

VERIFIED INVALID

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.
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
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
Status: NEW → ASSIGNED
Target Milestone: --- → M17
rginda: is this related to jst's changes for namespaces? (have those even gone in yet?) I think your JS should be using createElementNS()...
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.
That seems to fix it.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → INVALID
Adding 'verifyme' keyword
Keywords: verifyme
verifying bug
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: