Closed Bug 1305 Opened 26 years ago Closed 26 years ago

W3C DOM syntax: "setDOMAttribute" vs. "setAttribute" problem

Categories

(Core :: DOM: Core & HTML, defect, P2)

Product:
Core
Component:
DOM: Core & HTML
Platform:
x86
Windows 95
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: angus, Assigned: vidur)

Details

It's a minor syntax incompatibility issue. Here's the simple test case: <html> <head> <script> // this sets the attribute in the DOM but is non-standard: document.getElementById("mydiv").setDOMAttribute("ALIGN","right"); // this is the specified API; but it doesn't set the attribute in NGLayout: document.getElementById("mydiv").setAttribute("ALIGN",right); </script> </head> <body onload="init()"> <div id="mydiv"> Hello, world </div>
Status: NEW → ASSIGNED
I've put in a temporary fix, but I'll leave this bug open until the more permanent one goes in.
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Don't know why this wasn't closed a long time ago. Needless to say, it's fixed.
QA Contact: 4015 → 3847
Status: RESOLVED → REOPENED
Build 1999-06-02-08-M7 I can't get either document.getElementById("mydiv").setDOMAttribute("ALIGN","right"); or document.getElementById("mydiv").setAttribute("ALIGN",right); to work.
Status: REOPENED → RESOLVED
Closed: 26 years ago → 26 years ago
Try document.getElementById("mydiv").setAttribute("ALIGN","right"); See the quotes around the "right". Also, if layout doesn't change it's a different problem. This bug should only be reopened if we get a JavaScript error saying that setAttribute doesn't exist.
Status: RESOLVED → VERIFIED
Ok, Thanks Vidur! Verified build 1999-06-15-16-M7
denying in-testsuite. we have coverage on this.
Flags: in-testsuite-
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.