From d2bf50c63d4390f4b73a65d6ec620e1ad842fa47 Mon Sep 17 00:00:00 2001 From: SanderSpies Date: Thu, 19 Sep 2013 00:55:10 +0200 Subject: [PATCH] Give the user a warning when using unoptimized JSX code and send the user to the correct location in the documentation to optimize. --- docs/docs/08-tooling-integration.md | 2 +- vendor/browser-transforms.js | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/docs/docs/08-tooling-integration.md b/docs/docs/08-tooling-integration.md index abe17706c9..4733f36ec3 100644 --- a/docs/docs/08-tooling-integration.md +++ b/docs/docs/08-tooling-integration.md @@ -20,7 +20,7 @@ We provide CDN-hosted versions of React [on our download page](/react/downloads. We have instructions for building from `master` [in our GitHub repository](https://github.com/facebook/react). We build a tree of CommonJS modules under `build/modules` which you can drop into any environment or packaging tool that supports CommonJS. -## JSX +## JSX ### In-browser JSX Transform diff --git a/vendor/browser-transforms.js b/vendor/browser-transforms.js index b4ba305300..76bd7e0fd6 100644 --- a/vendor/browser-transforms.js +++ b/vendor/browser-transforms.js @@ -51,6 +51,7 @@ var load = exports.load = function(url, callback) { var xhr; xhr = window.ActiveXObject ? new window.ActiveXObject('Microsoft.XMLHTTP') : new XMLHttpRequest(); + // Disable async since we need to execute scripts in the order they are in the // DOM to mirror normal script loading. xhr.open('GET', url, false); @@ -79,6 +80,10 @@ runScripts = function() { return script.type === 'text/jsx'; }); + if(jsxScripts.length){ + console.warn("You are using unoptimized JSX (to optimize for production see: http://facebook.github.io/react/docs/tooling-integration.html#jsx)"); + } + jsxScripts.forEach(function(script) { if (script.src) { load(script.src);