Fix problems with annotation-xml in IE (work around IE lack of importNode, and bug with copying style attributes

Author: dpvc

config/Accessible-full.js

@@ -124,13 +124,14 @@ MathJax.Extension.mml2jax = {
       // In IE, outerHTML doesn't properly quote attributes, so quote them by hand
       // In Opera, HTML special characters aren't quoted in attributes, so quote them
       html = "<"+node.nodeName.toLowerCase();
-      var attributes = node.attributes;
-      for (i = 0, m = attributes.length; i < m; i++) {
-        if (attributes[i].specified) {
-          // Opera 11.5 beta turns xmlns into xmlns:xmlns, so put it back
-          // *** FIXME:  Similar problems may occur for other attributes?
-          html += " "+attributes[i].nodeName.toLowerCase().replace(/xmlns:xmlns/,"xmlns")+"=";
-          html += '"'+this.quoteHTML(attributes[i].nodeValue)+'"';
+      for (i = 0, m = node.attributes.length; i < m; i++) {
+        var attribute = node.attributes[i];
+        if (attribute.specified) {
+          // Opera 11.5 beta turns xmlns into xmlns:xmlns, so put it back (*** check after 11.5 is out ***)
+          html += " "+attribute.nodeName.toLowerCase().replace(/xmlns:xmlns/,"xmlns")+"=";
+          var value = attribute.nodeValue; // IE < 8 doesn't properly set style by setAttributes
+          if (value == null && attribute.nodeName === "style" && node.style) {value = node.style.cssText}
+          html += '"'+this.quoteHTML(value)+'"';
         }
       }
       html += ">";
@@ -153,6 +154,7 @@ MathJax.Extension.mml2jax = {
     return html;
   },
   quoteHTML: function (string) {
+    if (string == null) {string = ""}
     return string.replace(/&/g,"&#x26;").replace(/</g,"&lt;").replace(/>/g,"&gt;").replace(/"/g,"&quot;");
   },
 

config/Accessible.js

@@ -1183,10 +1183,38 @@ MathJax.ElementJax.mml.Augment({
     },
     Append: function () {
       for (var i = 0, m = arguments.length; i < m; i++) {
-        this.data.push(arguments[i]);
-        this.div.appendChild(document.importNode(arguments[i],true));
+        var node = this.Import(arguments[i]);
+        this.data.push(node);
+        this.div.appendChild(node);
       }
     },
+    Import: function (node) {
+      if (document.importNode) {return document.importNode(node,true)}
+      //
+      //  IE < 9 doesn't have importNode, so fake it.
+      //
+      var nNode, i, m;
+      if (node.nodeType === 1) { // ELEMENT_NODE
+        nNode = document.createElement(node.nodeName);
+        if (node.className) {nNode.className=iNode.className}
+        for (i = 0, m = node.attributes.length; i < m; i++) {
+          var attribute = node.attributes[i];
+          if (attribute.specified && attribute.nodeValue != null && attribute.nodeValue != '')
+            {nNode.setAttribute(attribute.nodeName,attribute.nodeValue)}
+          if (attribute.nodeName === "style") {nNode.style.cssText = attribute.nodeValue}
+        }
+        if (node.className) {nNode.className = node.className}
+      } else if (node.nodeType === 3 || node.nodeType === 4) { // TEXT_NODE or CDATA_SECTION_NODE
+        nNode = document.createTextNode(node.nodeValue);
+      } else if (node.nodeType === 8) { // COMMENT_NODE
+        nNode = document.createComment(node.nodeValue);
+      } else {
+        return document.createTextNode('');
+      }
+      for (i = 0, m = node.childNodes.length; i < m; i++)
+        {nNode.appendChild(this.Import(node.childNodes[i]))}
+      return nNode;
+    },
     value: function () {return this.div},
     toString: function () {return this.div.innerHTML}
   });