Changeset 8313

Timestamp:

04/26/07 23:53:44 (21 months ago)

Author:

bill

Message:

Filed bug #2855 that dojo.style("cssText") doesn't work on opera;
removed comments from code.

(Refs #2855)

Location:

dijit/trunk

Files:

• base/TemplatedWidget.js (modified) (3 diffs)
• form/_Spinner.js (modified) (1 diff)

dijit/trunk/base/TemplatedWidget.js

@@ -99 +99 @@
 
                         if(this.srcNodeRef){
-                                dojo.style(node, "cssText", this.srcNodeRef.style.cssText); // will fail on Opera?
+                                dojo.style(node, "cssText", this.srcNodeRef.style.cssText);
                                 node.className += " " + this.srcNodeRef.className;
                         }
@@ -295 +295 @@
 
         var tn = dojo.doc.createElement("div");
-        // dojo.style(tn, "display", "none");
-        dojo.style(tn, "visibility", "hidden");
+        tn.style.visibility="hidden";
         dojo.body().appendChild(tn);
         var tableType = "none";
@@ -324 +323 @@
         }
 
-        /* this doesn't make much sense, I'm assuming it just meant trim() so wrap was replaced with trim
-        if(wrap){
-                var ret = [];
-                // start hack
-                var fc = tn.firstChild;
-                ret[0] = ((fc.nodeValue == " ")||(fc.nodeValue == "\t")) ? fc.nextSibling : fc;
-                // end hack
-                // dojo.style(tn, "display", "none");
-                dojo.body().removeChild(tn);
-                return ret;
-        }
-        */
         var nodes = [];
         for(var x=0; x<parent.childNodes.length; x++){
                 nodes.push(parent.childNodes[x].cloneNode(true));
         }
-        dojo.style(tn, "display", "none"); // FIXME: why do we do this?
 //PORT  dojo.html.destroyNode(tn); FIXME: need code to prevent leaks and such
         dojo.body().removeChild(tn);

dijit/trunk/form/_Spinner.js

@@ -86 +86 @@
                         // textbox and domNode get the same style but the css separates the 2 using !important
                         if(this.srcNodeRef){
-                                dojo.style(node, "cssText", this.srcNodeRef.style.cssText); // will fail on Opera?
+                                dojo.style(node, "cssText", this.srcNodeRef.style.cssText);
                                 node.className += " " + this.srcNodeRef.className;
                         }