maxjhandsome
/
react
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #484 from syranide/jsxie8 

IE8 support for JSXTransformer
This commit is contained in:
Paul O’Shannessy 2013-11-08 14:38:51 -08:00
parent f7103a8629 595b482478
commit 91f6684fbf
2 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
package.json
View File

@ -37,8 +37,8 @@
},
"dependencies": {
"commoner": "~0.8.7",
"esprima-fb": "~1001.1001.2000-dev-harmony-fb",
"jstransform": "~1.0.1"
"esprima-fb": "~2001.1001.0-dev-harmony-fb",
"jstransform": "~2.0.1"
},
"devDependencies": {
"browserify": "~2.34.1",

16
vendor/browser-transforms.js vendored
View File

@ -43,7 +43,7 @@ var run = exports.run = function(code) {
var functionBody = jsx ? transform(code).code : code;
var scriptEl = document.createElement('script');
scriptEl.innerHTML = functionBody;
scriptEl.text = functionBody;
headEl.appendChild(scriptEl);
};
@ -75,11 +75,15 @@ var load = exports.load = function(url, callback) {
runScripts = function() {
var scripts = document.getElementsByTagName('script');
scripts = Array.prototype.slice.call(scripts);
var jsxScripts = scripts.filter(function(script) {
return script.type === 'text/jsx';
});
// Array.prototype.slice cannot be used on NodeList on IE8
var jsxScripts = [];
for (var i = 0; i < scripts.length; i++) {
if (scripts.item(i).type === 'text/jsx') {
jsxScripts.push(scripts.item(i));
}
}
console.warn("You are using the in-browser JSX transformer. Be sure to precompile your JSX for production - http://facebook.github.io/react/docs/tooling-integration.html#jsx");
jsxScripts.forEach(function(script) {