-
Notifications
You must be signed in to change notification settings - Fork 45.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update examples for Babel, 0.14; remove 3rd party code #4914
Conversation
@@ -11,23 +11,25 @@ | |||
<p> | |||
If you can see this, React is not running. Try running: | |||
</p> | |||
<pre>npm install -g react-tools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was wrong with this? Your way gives extra whitespace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing I just wanted prettier HTML. Comparing styling now though, looks like leaving as purely pre
will be better.
👍 |
Update examples for Babel, 0.14; remove 3rd party code
There are a couple changes here. The most important are that this brings us up to compatibility with 0.14 (mostly just using
ReactDOM
where appropriate). The other thing is that we stop trying to use JSXTransformer, which we aren't even building so these are all broken right now. And lastly, I took this opportunity to stop bundling 3rd party code (es5-shim, sham, console, jquery, bootstrap) and just point at CDNs. This does mean that the examples will require an internet connection to actually run but I think that's ok - I don't care about the airplane case as much anymore.I left the server-rendering example alone because I think I want to delete it. It's confusing to follow and run, and I think there are enough resources around on server rendering.